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

Boudewijn Rempt <b...@valdyas.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/kde/ |https://invent.kde.org/kde/
                   |krita/commit/b8870d02392a65 |krita/commit/dc6bf17a96556c
                   |ee5b4607a3df8bf0f7da363f08  |930f63e4cfbd42591f79987573

--- Comment #20 from Boudewijn Rempt <b...@valdyas.org> ---
Git commit dc6bf17a96556c930f63e4cfbd42591f79987573 by Boudewijn Rempt, on
behalf of Dmitry Kazakov.
Committed on 31/07/2019 at 08:39.
Pushed by rempt into branch 'krita/4.2'.

Make KisShortcutMatcher be more tolerant to recursive entering

Some tools, e.g. text tool, may open a dialog right inside
beginPrimaryAction() callback. It may make KisShortcutMatcher crazy,
because all the modal dialogs have hey own event loop, therefore
the state machine of the shortcuts matcher will be broken.

This patch makes the matcher a bit more tolerant to such faults. It
just manually checks if it is entered more that once, and resets the
action in case a recursion is detected.
Related: bug 408826, bug 409275

M  +168  -30   libs/ui/input/kis_shortcut_matcher.cpp
M  +1    -0    libs/ui/input/kis_shortcut_matcher.h

https://invent.kde.org/kde/krita/commit/dc6bf17a96556c930f63e4cfbd42591f79987573

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

Reply via email to