New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add 'Redo (un-undo)' function #2221
Conversation
Fixed minor bug and tested handling of each history event, so I publish this PR for review. |
I assigned this shortcut based on Apple's Human Interface Guideline, and Ubuntu's manual. Which keyboard shortcut is better for redo function? We could assign both of shortcuts for redo (add each definitions in keymap.json), but this is somewhat confusing. |
packages/node_modules/@node-red/editor-client/src/js/keymap.json
Outdated
Show resolved
Hide resolved
packages/node_modules/@node-red/editor-client/src/js/history.js
Outdated
Show resolved
Hide resolved
In Chrome and Safari on macOS, |
After you opened this, an issue was spotted and fixed in the code to undo 'creating subflow from selection'. The same fix was needed into the inverse event you had introduced - which I have applied here: c8acc6a Thanks! |
Proposed changes
This PR adds a 'redo' function to the Node-RED flow editor.
By this function, developers can cancel an undo operation.
Keyboard shortcut of redo is assigned to 'Ctrl-Shift-Z'.
To discuss implementation details, I opened this PR as a draft at first.
Checklist
grunt
to verify the unit tests pass