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.