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

Macro: When monomers are selected 'Del' and 'Backspace' hotkey delete selected monomers and switches to Erase tool #4028

Closed
Zhirnoff opened this issue Feb 5, 2024 · 2 comments · Fixed by #4385 or #4397

Comments

@Zhirnoff
Copy link
Collaborator

Zhirnoff commented Feb 5, 2024

Steps to Reproduce

  1. Switch to Macro mode
  2. Add some monomers on canvas
  3. Select with 'Selection tool' some monomers and press 'Del' or 'Backspace'

Actual behavior
When monomers are selected 'Del' and 'Backspace' hotkey delete selected monomers and switches to Erase tool

Expected behavior
When monomers are selected, pressing 'Del' or 'Backspace' hotkey deletes selected monomers without switching to the Erase tool. Should work like in Micro mode.

Desktop (please complete the following information):

  • OS: Windows 11
  • Browser Chrome
  • Version 112.0.5615.138 (Official Build) (64-bit)

Ketcher version
[Version 2.18.0-rc.3]

@Zhirnoff Zhirnoff added this to the Macromolecules Backlog milestone Feb 5, 2024
@Zhirnoff Zhirnoff changed the title Macro: The 'Del' and 'Backspace' hotkey does not delete monomer when hovering and pressing, but only switches to Erase tool Macro: When monomers are selected 'Del' and 'Backspace' hotkey delete selected monomers and switches to Erase tool Feb 15, 2024
@Zhirnoff
Copy link
Collaborator Author

Bug prioritized as requested @vanoprenko

tmrr13 added a commit that referenced this issue Apr 1, 2024
…lete selected monomers and switches to Erase tool
tmrr13 added a commit that referenced this issue Apr 1, 2024
…lete selected monomers and switches to Erase tool
NataliaLoginova added a commit that referenced this issue Apr 1, 2024
…ted-del-and-backspace-hotkey-delete-selected-monomers-and-switches-to-erase-tool

#4028 - Macro: When monomers are selected Del and Backspace hotkey delete selected monomers and switches to Erase tool
NataliaLoginova added a commit that referenced this issue Apr 2, 2024
…ted-del-and-backspace-hotkey-delete-selected-monomers-and-switches-to-erase-tool

#4028 - Macro: When monomers are selected Del and Backspace hotkey delete selected monomers and switches to Erase tool
NataliaLoginova added a commit that referenced this issue Apr 3, 2024
…ted-del-and-backspace-hotkey-delete-selected-monomers-and-switches-to-erase-tool

Backmerge: #4028 - Macro: When monomers are selected Del and Backspace hotkey delete selected monomers and switches to Erase tool
@AlexeyGirin
Copy link
Collaborator

Fixed.
Ketcher Version 2.21.0-rc.1 Build at 2024-04-15; 20:00:15
Indigo Version 1.20.0-rc.1.0-g8e8ffc3c3-wasm32-wasm-clang-12.0.0

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