-
Notifications
You must be signed in to change notification settings - Fork 66
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 hotkeys for undo and redo #1576
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please take care of these comments.
core/new-gui/src/app/workspace/component/navigation/navigation.component.ts
Outdated
Show resolved
Hide resolved
core/new-gui/src/app/workspace/component/navigation/navigation.component.ts
Outdated
Show resolved
Hide resolved
core/new-gui/src/app/workspace/component/navigation/navigation.component.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please address the comments and make some refactoring to improve readability.
core/new-gui/src/app/workspace/component/workflow-editor/workflow-editor.component.ts
Outdated
Show resolved
Hide resolved
core/new-gui/src/app/workspace/component/workflow-editor/workflow-editor.component.ts
Outdated
Show resolved
Hide resolved
core/new-gui/src/app/workspace/component/workflow-editor/workflow-editor.component.ts
Outdated
Show resolved
Hide resolved
core/new-gui/src/app/workspace/component/workflow-editor/workflow-editor.component.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor change needed and then it's good to be merged.
core/new-gui/src/app/workspace/component/workflow-editor/workflow-editor.component.ts
Outdated
Show resolved
Hide resolved
private _handleKeyboardAction(event: any) { | ||
this._onProcessKeyboardActionObservable = new Subject(); | ||
event.preventDefault(); | ||
this.workflowVersionService |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this workflowversion preview in this logic?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is listening for when workflow version preview is opened, do not enable undo/redo hotkey.
This reverts commit 70278c5.
The PR fixes a bug introduced by #1576, which blocks all other keyboard events other than undo/redo hotkeys.
This PR adds hotkeys for undo and redo button, and addresses issue #1553.
Before:
After: