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

--- Comment #2 from Dmitry Kazakov <dimul...@gmail.com> ---
Git commit 21889437905f05266092c854beb3282dd5f4b7a4 by Dmitry Kazakov.
Committed on 30/07/2019 at 14:16.
Pushed by dkazakov into branch 'master'.

Make Move Tool asynchronous

Calling blockUntilOperationsFinished() from tool event handler makes
KisShortcutMatcher to be entered recursively. It basically breaks events
flow and makes KisShortcutMatcher crazy.

I tried to move blocking into the shortcuts matcher itself (so that
it could treat recursive entrance correctly), but it doesn't handle
special cases like KisToolTransform::activate(), which should also
block (if we keep them blocking).

M  +9    -1    libs/ui/tool/strokes/move_stroke_strategy.cpp
M  +6    -1    libs/ui/tool/strokes/move_stroke_strategy.h
M  +1    -0    plugins/tools/basictools/CMakeLists.txt
A  +60   -0    plugins/tools/basictools/KisMoveBoundsCalculationJob.cpp    
[License: GPL (v2+)]
A  +46   -0    plugins/tools/basictools/KisMoveBoundsCalculationJob.h    
[License: GPL (v2+)]
M  +54   -38   plugins/tools/basictools/kis_tool_move.cc
M  +4    -1    plugins/tools/basictools/kis_tool_move.h
M  +4    -1   
plugins/tools/basictools/strokes/move_selection_stroke_strategy.cpp
M  +8    -1   
plugins/tools/basictools/strokes/move_selection_stroke_strategy.h

https://invent.kde.org/kde/krita/commit/21889437905f05266092c854beb3282dd5f4b7a4

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

Reply via email to