Description
There is a concurrency between Blockly undo management and browser undo management.
This affects safari and google chrome but does not affect firefox at least on OS X.
Take the fixed blockly demo at blockly/demos/fixed/index.html and add a textarea. If you perform changes in both the text area and the workspace, undo/redo with key shortcut merges Blockly undo management and browser undo management.
Here are the steps:
- load the modified page
- enter some text in the text area
- drag some block in the workspace
- trigger an undo event with the key shortcut
Problem: both the text area and the workspace are reverted during the same operation.
Expected behaviour: when the workspace has the focus, the undo/redo actions should apply to the workspace only.
Ideally, when the workspace has the focus, the menu item undo/redo action of the browser should apply to the workspace only.
Newer repro steps (2023/11/21)
- Run
npm run startto open the playground. - Type some text into the text field on the left.
- Drag a block into the workspace.
- Trigger an undo with a keyboard shortcut.
- Observe how the block is deleted and the text change is reverted. Expect the text to not be modified.
To Fix
Edit the keyboard shortcuts for undo and redo to call preventDefault on the event they are passed. You will need to add an extra event parameter to the callback method. Check out the type definition for keyboard shortcuts.