STEPS TO REPRODUCE 1. Press next or previous key button when there's two keys ahead or behind. 2. Press space to try to pan when mouse is over canvas. 3. The button will reactivate, causing it to jump ahead another frame. EXPECTED RESULT No extra frames should be jumped ahead or behind. Space key should not retrigger toolbutton.
This issue or at least related issues have kinda existed for a while, could be a duplicate of https://bugs.kde.org/show_bug.cgi?id=431682 and/or https://bugs.kde.org/show_bug.cgi?id=440399 (could be a duplicate of the former)
Yeah this is an issue in multiple parts of Krita for sure. I just tested it around the place and it's definitely an issue. However, in this case, we can solve this a little more easily since we can just remove the focus taking element of these QToolButtons. Since we already have hotkeys for people who want key-driven behavior, we can just ignore keyboard focus altogether. So I have a quick fix ready for this specific issue, but it's really just kicking the can down the road atm.
Git commit de5fdf8ba4cbab6d84abbbd019617adef2863ec6 by Eoin O'Neill. Committed on 25/10/2021 at 22:00. Pushed by eoinoneill into branch 'master'. Removed focus from KisTransportControl's QToolButtons. Should solve some usability issues when navigating keyframes where pressing space to pan the canvas would cause an erroneous next key / prev key call (or other transport button actions.) Since we have hotkeys to allow the user to trigger these based on key presses, we can simply remove this behavior altogether. However, this issue exists in other parts of Krita and will need a more robust solution to solve the issue globally across all widgets. M +6 -0 libs/ui/widgets/kis_transport_controls.cpp https://invent.kde.org/graphics/krita/commit/de5fdf8ba4cbab6d84abbbd019617adef2863ec6
Git commit 84eaf3c8672d53bab461ce8d96e90816392eb669 by Eoin O'Neill. Committed on 26/10/2021 at 20:38. Pushed by eoinoneill into branch 'krita/5.0'. Removed focus from KisTransportControl's QToolButtons. Should solve some usability issues when navigating keyframes where pressing space to pan the canvas would cause an erroneous next key / prev key call (or other transport button actions.) Since we have hotkeys to allow the user to trigger these based on key presses, we can simply remove this behavior altogether. However, this issue exists in other parts of Krita and will need a more robust solution to solve the issue globally across all widgets. (cherry picked from commit de5fdf8ba4cbab6d84abbbd019617adef2863ec6) M +6 -0 libs/ui/widgets/kis_transport_controls.cpp https://invent.kde.org/graphics/krita/commit/84eaf3c8672d53bab461ce8d96e90816392eb669