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
Confirmed on KDE Neon with current master.
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
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