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.

Sign up for GitHub

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

Jump to bottom

Macro: Hot keys for toolbar buttons are not implemented in Macromoleculas view #3713

Closed
AlexeyGirin opened this issue Dec 18, 2023 · 2 comments · Fixed by #3836
Closed

Macro: Hot keys for toolbar buttons are not implemented in Macromoleculas view #3713

AlexeyGirin opened this issue Dec 18, 2023 · 2 comments · Fixed by #3836
Assignees
@rrodionov91
Labels
feature request Macromolecules Priority: High
Milestone
Ketcher 2.18.0-rc.1

Comments

@AlexeyGirin
Copy link
Collaborator

AlexeyGirin commented Dec 18, 2023 •
edited by rrodionov91
Loading

Background
At Molecular view we are able to use key bindings for every toolbar button - ctrl+z, ctrl+y and etc.
image
Every button has hot key assigned and tooltip

Solution
We have to have the same on Macro view

Clear Canvas: Ctrl+Del, Ctrl+Backspace
Undo: Ctrl+Z
Redo: Ctrl+Shit+Z, Ctrl+Y
Erase: Del, Backspace
Select Rectangle: Shift+Tab, Esc
Zoom In: Ctrl+=
Zoom Out: Ctrl+_
Reset Zoom: Ctrl+0
"Select All" operation (not a button): Ctrl+A
@rrodionov91 rrodionov91 assigned Balzamova and unassigned Nitvex Dec 26, 2023
@Zhirnoff
Copy link
Collaborator

Zhirnoff commented Jan 3, 2024

Tests added to automation table.

Balzamova added a commit that referenced this issue Jan 3, 2024
@Balzamova
#3713 Hot keys for toolbar buttons for macromolecules
a3fb4fa
@rrodionov91 rrodionov91 assigned rrodionov91 and unassigned Balzamova Jan 4, 2024
@rrodionov91 rrodionov91 linked a pull request Jan 4, 2024 that will close this issue
Macro: #3713 - Hot keys for toolbar buttons are not implemented in Macromoleculas view #3836
Merged
9 tasks
rrodionov91 added a commit that referenced this issue Jan 5, 2024
@rrodionov91 @Balzamova
Macro: #3713 - Hot keys for toolbar buttons are not implemented in Ma…
0d31614 
…cromoleculas view (#3836)

Co-authored-by: Anastasiia.Balzamova <balzam0401@gmail.com>
Co-authored-by: Roman Rodionov <roman_rodionov@epam.com>
@Zhirnoff
Copy link
Collaborator

Tested. Some hotkeys only work in micro mode or macro mode. We need consistency. An additional bug has been created.
#3917

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rrodionov91 rrodionov91

Labels
feature request Macromolecules Priority: High
Projects
None yet
Milestone
Ketcher 2.18.0-rc.1
Development

Successfully merging a pull request may close this issue.

Macro: #3713 - Hot keys for toolbar buttons are not implemented in Macromoleculas view epam/ketcher
5 participants
@rrodionov91 @AlexeyGirin @Nitvex @Zhirnoff @Balzamova