Bug 296786 - Zoom to 100% pans to the top-left corner
Summary: Zoom to 100% pans to the top-left corner
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Preview-Image (show other bugs)
Version: 2.5.0
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-26 01:17 UTC by DrSlony
Modified: 2012-09-23 15:08 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 3.0.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description DrSlony 2012-03-26 01:17:00 UTC
Tested in 2.5.0. Zooming to 100% pans to the top-left corner. As most photographs have the subject near the center, the result is that when going through a few hundred photos one must keep doing mouse-strokes after zooming to 100% to get the center in view. This gets very tedious very quickly. Please make zooming to 100% focus on the center of the image (unless you zoomed into some area of a photo and then opened the next photo - in this case don't pan to the center, this behaviour is fine in 2.5.0).
Comment 1 Marcel Wiesweg 2012-07-08 12:09:10 UTC
Git commit c778bc15246604ad122b1e3e6817048b95e90244 by Marcel Wiesweg.
Committed on 08/07/2012 at 14:08.
Pushed by mwiesweg into branch 'master'.

When zooming by setting the zoom rate (slider), use the current view center as zoom anchor, not top-left.

M  +2    -1    NEWS
M  +3    -2    digikam/views/stackedview.cpp

http://commits.kde.org/digikam/c778bc15246604ad122b1e3e6817048b95e90244
Comment 2 DrSlony 2012-07-11 19:50:45 UTC
Thank you!
Comment 3 DrSlony 2012-07-11 19:53:07 UTC
"When zooming by setting the zoom rate (slider)"
Will this not work if I zoom using my own shortcut key "*" (numpad) to jump to 1:1? I would like it to, that's what I most often use, and it would be inconsistent if zooming to the center instead of the top-left only worked for one zooming method and not all methods.
Comment 4 DrSlony 2012-09-10 18:58:06 UTC
In 2.9.0 zooming using the slider zooms into the center (great), but using the shortcut key it still zooms into the top-left corner, so this fix would need to be extended to the shortcut zoom too.

Not completely resolved yet.
Comment 5 Marcel Wiesweg 2012-09-23 14:34:31 UTC
Git commit 466a690a843ea9cb6594fb54f1d02676b2d92438 by Marcel Wiesweg.
Committed on 23/09/2012 at 16:32.
Pushed by mwiesweg into branch 'master'.

Always default to image center as anchor for zooming.

The method which implements anchoring becomes the actual implementation method,
which means there's not zooming without anchoring anymore, and default is QPoint()
which is the image center.

M  +12   -12   libs/widgets/graphicsview/previewlayout.cpp
M  +2    -2    libs/widgets/graphicsview/previewlayout.h

http://commits.kde.org/digikam/466a690a843ea9cb6594fb54f1d02676b2d92438