epam / ketcher

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

In the “Modify RNA Builder” mode after adding phosphate to nucleosides in RNA Builder, in the Sequence mode the order of nucleotides is disrupted #4892

Open ViktoriaTkacheva opened 5 months ago

ViktoriaTkacheva commented 5 months ago

Steps to Reproduce

  1. Switch to the Macro mode
  2. Draw such a sequence of nucleosides, but it is important to manually drag each bond from AP R1 to AP R2 (do not use the “Select Connection Points” window)

    image
  3. Switch to the Sequence mode and select all letters

    image
  4. Right-click on them and choose the “Modify in RNA Builder” option

    image
  5. Select any phosphate in the RNA Builder
  6. Click on the “Update” button and choose the “Yes” option

Actual behavior The order of nucleotides is disrupted

image

Expected behavior After adding phosphate, the order of nucleotides remains the same

Screenshots

https://github.com/epam/ketcher/assets/139227567/ea93bdae-b508-41c5-882a-6805434cc8d6

Desktop (please complete the following information):

Ketcher version 2.22.0-rc.7, Build at 2024-06-20; 18:48:41

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

AlexeyGirin commented 5 months ago

Verified. image

Ketcher Version 2.23.0-rc.1 Build at 2024-07-02; 08:57:41 Indigo Version 1.22.0-rc.1.0-g2bef0ce6c-wasm32-wasm-clang-19.0.0 Win 10 Chrome Version 126.0.6478.127 (Official Build) (64-bit)

ViktoriaTkacheva commented 5 months ago

Bug reproduced.

https://github.com/epam/ketcher/assets/139227567/bf10838b-e4cd-4bca-b7ab-75b1f89dd202

rrodionov91 commented 3 months ago

Does not reproduce anymore

ViktoriaTkacheva commented 3 months ago

The bug is reproduced

https://github.com/user-attachments/assets/1bb43bfd-7b7c-4f4a-b32e-2ae0bb4b0c61