[Digikam-devel] [Bug 178540] zoom in previewwidget has poor behaviour

[Marcel Wiesweg]

http://bugs.kde.org/show_bug.cgi?id=178540


Marcel Wiesweg marcel wiesweg gmx de changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |marcel.wiesweg at gmx.de




--- Comment #9 from Marcel Wiesweg <marcel wiesweg gmx de>  2009-01-08 22:59:39 ---
Well I'm afraid but I have to say that the current implementation works much
better for me than with the patch applied.
With the patch there is a tendency of the cursor to move top-left. E.g. if I
start in the bottom-right corner and zoom in and out ten or fifteen times, the
cursor finally ends up in the top-left corner. This does not happen with the
current implementation. Maybe there's a difference between Linux and Windows?

Apart from this, I get these compilation warnings:
/home/marcel/freshmeat/multimedia/kde4/src/extragear/graphics/digikam/libs/widgets/common/previewwidget.cpp:
In member function ‘void Digikam::PreviewWidget::setZoomFactor(double,
bool)’:                                                                       
/home/marcel/freshmeat/multimedia/kde4/src/extragear/graphics/digikam/libs/widgets/common/previewwidget.cpp:252:
warning: unused variable ‘oldZoom’  
/home/marcel/freshmeat/multimedia/kde4/src/extragear/graphics/digikam/libs/widgets/common/previewwidget.cpp:
In member function ‘virtual void
Digikam::PreviewWidget::contentsWheelEvent(QWheelEvent*)’:                    
/home/marcel/freshmeat/multimedia/kde4/src/extragear/graphics/digikam/libs/widgets/common/previewwidget.cpp:680:
warning: comparison between signed and unsigned integer expressions             
/home/marcel/freshmeat/multimedia/kde4/src/extragear/graphics/digikam/libs/widgets/common/previewwidget.cpp:
In member function ‘void Digikam::PreviewWidget::setZoomFactor(double,
bool)’:                                                                       
/home/marcel/freshmeat/multimedia/kde4/src/extragear/graphics/digikam/libs/widgets/common/previewwidget.cpp:253:
warning: ‘cpy’ may be used uninitialized in this function                   
/home/marcel/freshmeat/multimedia/kde4/src/extragear/graphics/digikam/libs/widgets/common/previewwidget.cpp:253:
warning: ‘cpx’ may be used uninitial


-- 
Configure bugmail: http://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.