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

Paste to canvas recogniton logic should be changed #5974

Closed
AlexeyGirin opened this issue Nov 14, 2024 · 0 comments · Fixed by #6017
Closed

Paste to canvas recogniton logic should be changed #5974

AlexeyGirin opened this issue Nov 14, 2024 · 0 comments · Fixed by #6017
Assignees
@mnmsvlw
Labels
bug Export to Sequence Bucket: Bugs related to Export to Sequence mode IDT Bucket: IDT related issues Sequence mode Bucket: Sequence mode related issues
Milestone
Ketcher 2.28.0-rc...

Comments

@AlexeyGirin
Copy link
Collaborator

AlexeyGirin commented Nov 14, 2024 •
edited
Loading

Steps to Reproduce

  1. Toggle to Macromolecules mode - Sequence mode
  2. Click on the canvas and paste from clipboard (pressing Ctrl+V) following IDT text: rArCrGrTrU

Actual behavior
System recognized IDT as Sequence
image

Expected behavior
System should recognize IDT first
image

Recognition order should be:

  1. 3-letter
  2. IDT
  3. 1-letter
  4. ... all the rest

Versions

  • Ketcher Version 2.27.0-rc.2 Build at 2024-11-12; 19:26:10
  • Indigo Toolkit Version 1.26.0-rc.2.0-gfec30ecd8-wasm32-wasm-clang-19.0.0
  • Chrome Version 131.0.6778.70 (Official Build) (64-bit)
  • Win10
@AlexeyGirin AlexeyGirin added bug IDT Bucket: IDT related issues Sequence mode Bucket: Sequence mode related issues Export to Sequence Bucket: Bugs related to Export to Sequence mode labels Nov 14, 2024
@mnmsvlw mnmsvlw self-assigned this Nov 25, 2024
mnmsvlw added a commit that referenced this issue Nov 26, 2024
@mnmsvlw
#5974 - Paste to canvas recogniton logic should be changed
0c8ac05
@mnmsvlw mnmsvlw linked a pull request Nov 26, 2024 that will close this issue
#5974 – Paste to canvas recogniton logic should be changed #6017
Merged
9 tasks
NataliaLoginova pushed a commit that referenced this issue Nov 27, 2024
@mnmsvlw
#5974 – Paste to canvas recogniton logic should be changed (#6017)
9987155 
* #5974 - Paste to canvas recogniton logic should be changed

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

@mnmsvlw mnmsvlw

Labels
bug Export to Sequence Bucket: Bugs related to Export to Sequence mode IDT Bucket: IDT related issues Sequence mode Bucket: Sequence mode related issues
Projects
None yet
Milestone
Ketcher 2.28.0-rc.1 (Current)
Development

Successfully merging a pull request may close this issue.

#5974 – Paste to canvas recogniton logic should be changed epam/ketcher
2 participants
@AlexeyGirin @mnmsvlw