Bug 455243 - Two finger drag "zoom and rotate" and one finger drag "pan canvas" does not continue seamlessly
Summary: Two finger drag "zoom and rotate" and one finger drag "pan canvas" does not c...
Status: CONFIRMED
Alias: None
Product: krita
Classification: Applications
Component: Shortcuts and Canvas Input Settings (other bugs)
Version First Reported In: nightly build (please specify the git hash!)
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2022-06-14 11:16 UTC by Alvin Wong
Modified: 2023-01-26 13:56 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alvin Wong 2022-06-14 11:16:41 UTC
SUMMARY


STEPS TO REPRODUCE
1. Set "Two Finger Drag" to "Zoom and Rotate Canvas" (Discrete Rotate Mode)
2. Remove "One Finger Drag" from "Alternate Invocation" (Sample Foreground Color from Merged Image)
3. Set "One Finger Drag" to "Pan Canvas" (Pan Mode)
4. Do one finger drag on canvas (it should begin panning)
5. Switch to two finger gesture by adding a second finger while still panning (it should switch to zoom and rotate mode seamlessly)
6. Try to switch to one finger drag by removing one finger while still panning

OBSERVED RESULT
The panning gesture stops.

EXPECTED RESULT
Canvas should continue panning with one finger.

SOFTWARE/OS VERSIONS
Windows: Windows 10
Comment 1 Alvin Wong 2022-06-16 10:35:14 UTC
I am marking this as a regression because this had been working with the old touch gesture implementation in 5.0 (when touch painting is disabled).
Comment 2 sh_zam 2022-06-21 20:47:05 UTC
Hello!

Well, this is something that I did intentionally. 

Let's say if we assign "One Finger Drag" to "Alternate Invocation (Sample Foreground Color from Merged Image)", then once the user lifts the finger (say in 2 finger drag) this alternate invocation would start immediately and can be annoying. Yes, we can add some slop before we start this action, but I thought disabling it altogether would be more preferable?
Comment 3 Alvin Wong 2022-06-22 08:19:44 UTC
Well, but I do not have colour sampler assigned to one finger drag. My point is that the combination of "pan canvas" and "zoom and rotate canvas" gestures is special in that it should turn the canvas into a direct manipulation surface. Switching between these two gestures should be seamless as any touchscreen users would expect, and like other applications out there.

For any other gestures, yes, not switching would be the logical choice.
Comment 4 wolthera 2023-01-26 13:56:02 UTC
Given sh_zam can reproduce this, I'll mark it as confirmed.