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

When mouse hovering on atom or bond hotkey CTRL+Z (Undo) is not working #2442

Closed
Zhirnoff opened this issue Apr 3, 2023 · 0 comments · Fixed by #2480
Closed

When mouse hovering on atom or bond hotkey CTRL+Z (Undo) is not working #2442

Zhirnoff opened this issue Apr 3, 2023 · 0 comments · Fixed by #2480
Assignees
Labels

Comments

@Zhirnoff
Copy link
Collaborator

Zhirnoff commented Apr 3, 2023

Steps to Reproduce

  1. Open Ketcher
  2. Draw 'Benzene'
  3. Draw any bonds on Benzene atoms
  4. Hover mouse cursor over any bond of 'Benzene' and press CTRL+Z (Undo)
  5. Hover mouse cursor over any atom of 'Benzene' and press CTRL+Z (Undo)

Note: When mouse hovering on atom or bond hotkeys for Clear canvas, Redo, Aromatize, Dearomatize, Layout, Clean Up, Calculate CIP don't work as well

Actual behavior
Nothing happens. When mouse hovering on atom or bond hotkey CTRL+Z (Undo) is not working

Expected behavior
When mouse hovering on atom or bond hotkey CTRL+Z (Undo) is working.

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser Chrome
  • Version 109.0.5414.120 (Official Build) (64-bit)

Ketcher version
Version 2.9.0-rc.4

@Zhirnoff Zhirnoff added the bug label Apr 3, 2023
@Zhirnoff Zhirnoff added this to the Ketcher 2.10.0-rc.1 milestone Apr 3, 2023
Nitvex pushed a commit that referenced this issue Apr 19, 2023
… working (#2480)

* #2442 - When mouse hovering on atom or bond hotkey undo is not working

* #2442 - add condition for handle hotkey over item

---------

Co-authored-by: Maxim Novikov <Maxim_Novikov@epam.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants