epam / ketcher

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

Macro: After merging several chains, when clicking the “Undo” button, the order of the chains is broken #4342

Open ViktoriaTkacheva opened 6 months ago

ViktoriaTkacheva commented 6 months ago

Steps to Reproduce

  1. Switch to the Macro mode - Sequence mode

  2. Add any five nucleotide chains to the canvas (for example, AAA, TTT, CCC, UUU, GGG)

    image
  3. Merge chains 1 and 2, and chains 3 and 4, using the “Backspace” button

    image
  4. Click the “Undo” button 2 times

Actual behavior The chains do not take the position, which they occupied before merging.

After the first ”Undo”:

image

After the second ”Undo”:

image

Expected behavior The order of the chains is not broken. The chains take the position, which they occupied before merging

image

Screenshots

https://github.com/epam/ketcher/assets/139227567/14c612f1-7d68-4bbb-a131-56fbde490e60

Desktop (please complete the following information):

Ketcher version 2.20.0-rc.1

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

AlexeyGirin commented 5 months ago

Moved to Refined Backlog since not fixed over 1 release