Bug 411507 - Zoom icon doesn’t update immediately when pressing ‘Ctrl’
Summary: Zoom icon doesn’t update immediately when pressing ‘Ctrl’
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Tools (show other bugs)
Version: git master (please specify the git hash!)
Platform: Other Linux
: NOR minor
Target Milestone: ---
Assignee: Dmitry Kazakov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-09-01 20:02 UTC by Karl Ove Hufthammer
Modified: 2020-06-19 14:41 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Karl Ove Hufthammer 2019-09-01 20:02:51 UTC
SUMMARY
When one uses the zoom tool and inverts the zoom mode using ‘Ctrl’, the zoom icon is only updated to the reflect the new zoom mode when one moves the mouse cursor. It should be updated when one actually changes the zoom mode.


STEPS TO REPRODUCE
1. Open an image.
2. Select the zoom tool.
3. Position the mouse cursor over the image.
4. Press ‘Ctrl’.
5. Move the cursor (while holding ‘Ctrl’ pressed).
6. Hold the cursor still and release ‘Ctrl’.
7. Move the cursor.

OBSERVED RESULT
In step 4, the cursor remains a ‘zoom in’ cursor. It is only changed in step 5.
In step 6, the cursor remains a ‘zoom out’ cursor. It is only changed in step 7.


EXPECTED RESULT
In step 4, the cursor should be changed to the ‘zoom out’ cursor.
In step 6, the cursor should be changed to the ‘zoom in’ cursor.


SOFTWARE/OS VERSIONS
Operating System: openSUSE Tumbleweed 20190829
KDE Plasma Version: 5.16.4
KDE Frameworks Version: 5.61.0
Qt Version: 5.13.0
Kernel Version: 5.2.10-1-default
OS Type: 64-bit
Comment 1 wolthera 2019-09-21 16:17:14 UTC
Confirmed on KDE Neon with current master.
Comment 2 Dmitry Kazakov 2020-06-19 14:41:11 UTC
Git commit 651b52340f1a36b1c2d7a7e8e404b50e14d95926 by Dmitry Kazakov.
Committed on 19/06/2020 at 14:40.
Pushed by dkazakov into branch 'krita/4.3'.

Fix handling of Control modifier in the Zoom Tool

We should call base class methods to make sure it baves correctly
(has lastMousePos value in sync)

M  +3    -5    libs/flake/tools/KoZoomTool.cpp

https://invent.kde.org/graphics/krita/commit/651b52340f1a36b1c2d7a7e8e404b50e14d95926
Comment 3 Dmitry Kazakov 2020-06-19 14:41:45 UTC
Git commit 7b3a9caae920208379f39eef14dc58dd1af832c4 by Dmitry Kazakov.
Committed on 19/06/2020 at 14:41.
Pushed by dkazakov into branch 'master'.

Fix handling of Control modifier in the Zoom Tool

We should call base class methods to make sure it baves correctly
(has lastMousePos value in sync)

M  +3    -5    libs/flake/tools/KoZoomTool.cpp

https://invent.kde.org/graphics/krita/commit/7b3a9caae920208379f39eef14dc58dd1af832c4