Skip to content
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

Implement shortcuts #606

Closed
romicolman opened this issue Oct 21, 2024 · 4 comments · Fixed by #612
Closed

Implement shortcuts #606

romicolman opened this issue Oct 21, 2024 · 4 comments · Fixed by #612
Assignees

Comments

@romicolman
Copy link
Collaborator

Description

Based on research conducted on ticket #597, please implement the following list of shortcuts:

  • Alt + M Toggle the metadata panel
  • Alt + R Toggle the report panel
  • Alt + S Toggle the source panel
  • Ctrl + Z Undo last change
  • Ctrl + Y Redo last change
  • Ctrl + Q Revert the changes
  • Ctrl + S Save the changes
  • Enter - Confirm action
  • Esc - Cancel action
@guergana
Copy link
Collaborator

Hi @romicolman I see a problem with Ctrl+Q, this is the default shortcut for closing applications. If you do it on Linux it closes the app. Maybe we can find another shortcut for reverting the changes?

@romicolman
Copy link
Collaborator Author

Hi! Let's implement the rest and omit the Ctrl+Q for now

@romicolman
Copy link
Collaborator Author

Hey! I tested it again today. Apart from ENTER and ESC, shortcuts do not work on Mac.

@guergana
Copy link
Collaborator

guergana commented Nov 7, 2024

Hey! I tested it again today. Apart from ENTER and ESC, shortcuts do not work on Mac.

We have decided to merge this as it is working for windows and Linux and will create a ticket for Mac in the future.

@github-project-automation github-project-automation bot moved this from Code review to Done in ODE: Stable Release Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants