[Bug 296786] New: Zoom to 100% pans to the top-left corner

[DrSlony-3]

https://bugs.kde.org/show_bug.cgi?id=296786

            Bug ID: 296786
          Severity: normal
           Version: 2.5.0
          Priority: NOR
          Assignee: [hidden email]
           Summary: Zoom to 100% pans to the top-left corner
    Classification: Unclassified
                OS: Linux
          Reporter: [hidden email]
          Hardware: Gentoo Packages
            Status: UNCONFIRMED
         Component: Preview
           Product: digikam

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).

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Marcel Wiesweg]

https://bugs.kde.org/show_bug.cgi?id=296786

Marcel Wiesweg <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|                            |http://commits.kde.org/digi
                   |                            |kam/c778bc15246604ad122b1e3
                   |                            |e6817048b95e90244

--- Comment #1 from Marcel Wiesweg <[hidden email]> ---
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

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

https://bugs.kde.org/show_bug.cgi?id=296786

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]
   Version Fixed In|                            |2.8.0

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[DrSlony-3]

https://bugs.kde.org/show_bug.cgi?id=296786

--- Comment #2 from DrSlony <[hidden email]> ---
Thank you!

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[DrSlony-3]

https://bugs.kde.org/show_bug.cgi?id=296786

--- Comment #3 from DrSlony <[hidden email]> ---
"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.

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[DrSlony-3]

https://bugs.kde.org/show_bug.cgi?id=296786

--- Comment #4 from DrSlony <[hidden email]> ---
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.

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Marcel Wiesweg]

https://bugs.kde.org/show_bug.cgi?id=296786

--- Comment #5 from Marcel Wiesweg <[hidden email]> ---
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

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

https://bugs.kde.org/show_bug.cgi?id=296786

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Version Fixed In|2.8.0                       |3.0.0

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel