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

Aromatizing/Dearomatizing changes molecula position (Layout applied?) #3830

Closed
AlexeyGirin opened this issue Jan 3, 2024 · 1 comment · Fixed by #3837
Closed

Aromatizing/Dearomatizing changes molecula position (Layout applied?) #3830

AlexeyGirin opened this issue Jan 3, 2024 · 1 comment · Fixed by #3837
Assignees
@Nitvex
Labels
bug Priority: Medium query features phase II
Milestone
Ketcher 2.18.0-rc.1

Comments

@AlexeyGirin
Copy link
Collaborator

AlexeyGirin commented Jan 3, 2024 •
edited
Loading

Steps to Reproduce

  1. Put Benzene on the canvas
  2. Press Aromatize button
  3. Press Dearomatize button
2024-01-03_21h14_12.mp4

Actual behavior
Molecule changes its position

Expected behavior
Molecule remain in place

Issue found while testing epam/Indigo#1446

Desktop (please complete the following information):

  • OS: Win10
  • Browser Chrome Version 120.0.6099.130 (Official Build) (64-bit)

Ketcher version [e.g. v2.4.2].
Ketcher Version 2.15.0-rc.4 Build at 2023-10-25; 14:15:43
Indigo Toolkit Version 1.18.0-dev.2.0-g369c82f8c-x86_64-linux-gnu-11.4.0
@AlexeyGirin AlexeyGirin added the bug label Jan 3, 2024
@AlexeyGirin AlexeyGirin added this to the Query Features - Phase II milestone Jan 3, 2024
Nitvex added a commit that referenced this issue Jan 4, 2024
@Nitvex
#3830 – Aromatizing/Dearomatizing changes molecula position
996a220
@Nitvex Nitvex mentioned this issue Jan 4, 2024
Merged
9 tasks
Nitvex added a commit that referenced this issue Jan 5, 2024
@Nitvex
#3830 – Aromatizing/Dearomatizing changes molecula position
0c0c505
Nitvex added a commit that referenced this issue Jan 5, 2024
@Nitvex
#3830 – fixing CI tests
91d6f1b
@AlexeyGirin AlexeyGirin mentioned this issue Jan 5, 2024
Closed
4 tasks
Nitvex added a commit that referenced this issue Jan 5, 2024
@Nitvex
#3830 – fixing CI tests
8eef8c6
Nitvex added a commit that referenced this issue Jan 5, 2024
@Nitvex
#3830 – fixing CI tests
19a5fc6
Nitvex added a commit that referenced this issue Jan 8, 2024
@Nitvex
#3830 – Aromatizing/Dearomatizing changes molecula position (#3837)
2c9945a 
* #3830 – Aromatizing/Dearomatizing changes molecula position

* #3830 – fixing CI tests
@AlexeyGirin
Copy link
Collaborator Author

Fixed.

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

@Nitvex Nitvex

Labels
bug Priority: Medium query features phase II
Projects
None yet
Milestone
Ketcher 2.18.0-rc.1
Development

Successfully merging a pull request may close this issue.

#3830 – Aromatizing/Dearomatizing changes molecula position epam/ketcher
4 participants
@rrodionov91 @vanoprenko @AlexeyGirin @Nitvex