[decision-fatigue-is-real]

Description

Synchronises the selection handler with the pointer shortcut system, so that detecting events from the pointer shortcut system does not cancel the scale tool when in operation.

Problem

Previously, binding an action to double/triple tap for touches in the input menu in settings, while the touch shortcut was bound to a specific toolbar position corresponding to the Select tool, caused selections to not be processed in a majority of cases.

This was likely due to the Select tool's scale events being processed initially, then being overwritten by the pointer shortcut manager, which checks for multiple taps and cancels the Select tool's selection for Lasso and Rectangle modes.

Related Issue

Closes #1345

[CodeDoctorDE]

I tested this with a left-button double-click shortcut and found two regressions:

• Short rectangle drags are ignored even when the left mouse button is set to Active tool.
• With Lasso active and the left mouse button mapped to Rectangle, a lasso outline can stay on the canvas after release.

I’d have the viewport cancel the pending click shortcut as soon as Flutter recognizes a drag, then replay the buffered pointer events before calling onScaleStart. That lets the mapped tool handle the whole gesture, including short drags below the shortcut manager’s movement threshold.

[decision-fatigue-is-real]

Thank you for your reply. I've now implemented the suggestions, and now both of those regressions no longer occur.

[CodeDoctorDE]

i would save _inputGeneration before await and then test if _isCurrentInput. Otherwise i think the method would still run after the page is closed

[decision-fatigue-is-real]

Thank you for spotting this issue. I've now added in the _isCurrentInput check.

[CodeDoctorDE]

Now its look good!
Thanks for the multiple contributons