epam / ketcher

Web-based molecule sketcher
https://lifescience.opensource.epam.com/ketcher/demo.html
Apache License 2.0
474 stars 164 forks source link

Do/Undo loginc doesn't save Flex to Sequence mode switch as an action and doesn't undo it #5068

Closed AlexeyGirin closed 2 months ago

AlexeyGirin commented 3 months ago

NOTE: Probably the same root cause with https://github.com/epam/ketcher/issues/5048 and https://github.com/epam/ketcher/issues/5049

Steps to Reproduce

  1. Go to Macromolecules - Flex mode <---Important!
  2. Load from file: Do-Undo loginc doesn't save Flex to Sequence mode switch as an action and doesn't undo it.zip image
  3. Delete some chains image
  4. Switch view to Sequence mode image
  5. Press Undo button

Actual behavior No switch to Flex mode happens, system restores deleted chains image

Expected behavior System switches to Flex mode image

Environment details:

Issue found while testing - https://github.com/epam/ketcher/issues/3532

svvald commented 2 months ago

Can't reproduce, probably fixed by https://github.com/epam/ketcher/issues/5049 as having common root cause

Zhirnoff commented 1 month ago

Tested. Bug fixed.

https://github.com/user-attachments/assets/c9a0d05d-7b7c-4a7f-9d33-8c628b941f04

Desktop:

Ketcher version [Version 2.24.0-rc.2] Indigo version [Version 1.23.0-rc.2]