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

Peter Mühlenpfordt <de...@ukn8.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|                            |https://commits.kde.org/gwe
                   |                            |nview/945c8a7a8d0b8326b194d
                   |                            |44ec004963c0694f49a
             Status|CONFIRMED                   |RESOLVED

--- Comment #7 from Peter Mühlenpfordt <de...@ukn8.de> ---
Git commit 945c8a7a8d0b8326b194d44ec004963c0694f49a by Peter Mühlenpfordt.
Committed on 16/07/2018 at 15:15.
Pushed by muhlenpfordt into branch 'master'.

Toggle between zoom options "Fit"/"Fill" and "100%" on multiple activation

Summary:
In previous versions of Gwenview it was possible to toggle between
{nav Fit} and {nav 100%} zoom factors using the shortcut {key F}.
After 3e10699ac37c added the {nav Fit Width} zoom option this feature
did not work any more.
This patch adds toggling for both {nav Fit} and {nav Fill} zoom
options if the shortcut {key F} or one of the buttons is pressed
multiple times. Zooming to 100% is centered to the current cursor
position like middle click works.

Test Plan:
# Open Gwenview in View Mode
# Use zoom buttons {nav Fit}/{nav Fill} or {key F} multiple times
# The zoom should toggle between {nav Fit} and {nav 100%},
  respectively between {nav Fill} and {nav 100%}

Reviewers: #gwenview, rkflx

Reviewed By: #gwenview, rkflx

Subscribers: rkflx

Differential Revision: https://phabricator.kde.org/D14093

M  +2    -0    lib/documentview/abstractimageview.cpp
M  +30   -0    lib/documentview/documentview.cpp
M  +2    -0    lib/documentview/documentview.h
M  +2    -7    lib/documentview/documentviewcontroller.cpp

https://commits.kde.org/gwenview/945c8a7a8d0b8326b194d44ec004963c0694f49a

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to