Bug 365933 - pan tool is broken in mirror view
Summary: pan tool is broken in mirror view
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Tools (show other bugs)
Version: 3.0
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Victor Wåhlström
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-07-21 10:15 UTC by codl
Modified: 2017-11-10 07:43 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description codl 2016-07-21 10:15:41 UTC
If i have mirror view enabled (by pressing M), then the pan tool zips around predictably, and the zoom tools acts as if mirror view was not enabled

Reproducible: Always

Steps to Reproduce:
1. press M to enable mirror view
2. try to pan with the pan tool, or zoom with the zoom tool
Comment 1 wolthera 2016-07-21 10:33:30 UTC
Confirmed here.
Comment 2 Nimthora 2016-07-21 10:33:51 UTC
Can confirm this happens on Ubuntu 15.10 with Krita master branch (git a8f901e40fb7de35a29a4b4803ccbacab802c8bd).
Comment 3 Halla Rempt 2016-08-13 09:15:52 UTC
Is this a regression caused by the relative zoom tool patch?
Comment 4 wolthera 2016-08-13 14:33:07 UTC
No, it's too old for that, and it only happens with the zoom/pan TOOLS, not the regular zoom/pan
Comment 5 Halla Rempt 2016-08-15 14:02:38 UTC
Ah, then that's probably because the zoom/pan tools actually predate the canvas mirror feature.
Comment 6 Victor Wåhlström 2017-06-20 19:45:02 UTC
Git commit aaaf1eea993d3dc65bcd3585cabf7751464e9567 by Victor Wåhlström.
Committed on 20/06/2017 at 19:43.
Pushed by victorw into branch 'master'.

Rewrote the Pan tool based on KisTool. Fixes pan issues when mirroring is enabled.

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

M  +0    -2    libs/flake/CMakeLists.txt
M  +24   -0    libs/flake/KoCanvasController.h
M  +20   -0    libs/flake/KoCanvasControllerWidget.cpp
M  +5    -0    libs/flake/KoCanvasControllerWidget.h
M  +0    -6    libs/flake/KoToolManager.cpp
M  +0    -3    libs/flake/KoToolRegistry.cpp
D  +0    -121  libs/flake/tools/KoPanTool.cpp
D  +0    -73   libs/flake/tools/KoPanTool.h
D  +0    -40   libs/flake/tools/KoPanToolFactory.cpp
D  +0    -35   libs/flake/tools/KoPanToolFactory.h
M  +1    -0    plugins/tools/basictools/CMakeLists.txt
M  +2    -0    plugins/tools/basictools/default_tools.cc
A  +100  -0    plugins/tools/basictools/kis_tool_pan.cpp     [License: GPL (v2+)]
A  +54   -0    plugins/tools/basictools/kis_tool_pan.h     [License: GPL (v2+)]

https://commits.kde.org/krita/aaaf1eea993d3dc65bcd3585cabf7751464e9567
Comment 7 Halla Rempt 2017-11-08 13:35:28 UTC
Victor, can we now close this bug?
Comment 8 Victor Wåhlström 2017-11-10 07:43:05 UTC
This is a two-for-one bug, and I only fixed the pan tool version of it. I created bug 386704 to track the zoom tool version of the bug. Unfortunately the same solution cannot be applied to the zoom tool, so it's not as easy.

I'll edit the title of this bug so it can be closed. The zoom tool bug should be tracked in bug 386704.