Holding down a shortcut invokes it repeatedly

[somebody1234]

e.g. Holding down Ctrl+V to paste a node.

This is not an issue in shortcuts.ts as it provides no event handling itself.

The issue is with using useEvent(window, 'keydown'. This task will involve adding a custom composable to events.ts which avoids re-triggering the callback until after the next keyup event. Care should be taken to ensure that the keyup event wasn't of the Ctrl being released (if that is possible).

[somebody1234]

On second thought it's possible to just check event.repeat: https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/repeat

So it may be possible to check for this in shortcuts.ts

MDN Web Docs

KeyboardEvent: repeat property - Web APIs | MDN

The repeat read-only property of the KeyboardEvent interface returns a boolean value that is true if the given key is being held down such that it is automatically repeating.

[farmaazon]

Cannot we just check repeat flag of keydown event?

[enso-bot]

Paweł Grabarz reports a new STANDUP for yesterday (2024-03-25):

Progress: Fixed repeated shortcut handling bug and adjusted visualization defaults. Cleaned up rust codebase to remove large amount of no longer needed but costly to build modules. It should be finished by 2024-03-25.