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

Pressing Clear Canvas with Reaction Arrow under mouse cursor causes errors in DevTool console #2483

Closed
Zhirnoff opened this issue Apr 13, 2023 · 1 comment · Fixed by #2561
Closed

Pressing Clear Canvas with Reaction Arrow under mouse cursor causes errors in DevTool console #2483

Zhirnoff opened this issue Apr 13, 2023 · 1 comment · Fixed by #2561
Labels
bug
Milestone
Ketcher 2.11.0-rc.1

Comments

@Zhirnoff
Copy link
Collaborator

Steps to Reproduce

  1. Open Ketcher and DevTool browser console
  2. Select Reaction Arrow and place on canvas
  3. With Selection Tool select Reaction Arrow and press CTRL+C
  4. Press CTRL+V and with Reaction Arrow under mouse cursor press Clear Canvas button

Actual behavior
Uncaught TypeError in DevTool console

Expected behavior
Reaction Arrow under mouse cursor and on canvas dissapears

Screenshots

2023-04-13_22h47_43.mp4

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.6
@Zhirnoff Zhirnoff added the bug label Apr 13, 2023
@Zhirnoff Zhirnoff added this to the Ketcher 2.10.0-rc.1 milestone Apr 13, 2023
@KonstantinEpam23
Copy link
Collaborator

If this will turn out too complex for a quick fix - move to Refined backlog

@ghost ghost assigned ghost and unassigned Nitvex May 1, 2023
ghost pushed a commit that referenced this issue May 1, 2023
@VandaEPAM
#2483 - Changed actions order in clear action
400364a
@ghost ghost linked a pull request May 1, 2023 that will close this issue
#2483: Pressing Clear Canvas with Reaction Arrow under mouse cursor causes errors in DevTool console #2561
Merged
Nitvex pushed a commit that referenced this issue May 5, 2023
#2483 - Changed actions order in clear action (#2561)
08c5f80
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
Ketcher 2.11.0-rc.1
3 participants
@Nitvex @Zhirnoff @KonstantinEpam23