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

Ketcher structure copy-paste in KET instead of MOL #6411

Closed
Tracked by #6600
Zhirnoff opened this issue Feb 3, 2025 · 1 comment · Fixed by #6420, #6424 or #6426
Closed
Tracked by #6600

Ketcher structure copy-paste in KET instead of MOL #6411

Zhirnoff opened this issue Feb 3, 2025 · 1 comment · Fixed by #6420, #6424 or #6426
Assignees
@rrodionov91
Labels
Priority: High Severity: High High level visibility to the user User`s Feedback
Milestone
Ketcher 3.0.0-rc.7

Comments

@Zhirnoff
Copy link
Collaborator

Zhirnoff commented Feb 3, 2025 •
edited
Loading

Summary:
When copying a structure from Ketcher and pasting it into ChemDraw, instead of the expected molecular structure, a Ketcher text string (KET file, JSON string) is pasted. In some cases, ChemDraw returns an error indicating that the text is too long.
Note: On the previous version of Ketcher it copied as MOL file

Steps to Reproduce

  1. Open Ketcher and draw any structure. (Benzene for example)
  2. Copy the structure using Ctrl+C
  3. Paste into Word for example by Ctrl+V
  4. Observe pasted format

Actual behavior
Ketcher structure copy-paste in KET instead of MOL

Expected behavior
Ketcher structure copy-paste in MOL

Desktop:

  • OS: Windows 11
  • Browser Chrome
  • Version 131.0.6778.205 (Official Build) (64-bit)

Ketcher version
[Version 3.0.0-rc.6]
Indigo version
[Version 1.28.0-rc.3]

#188852421
@Zhirnoff Zhirnoff added CDX-ChemDraw Bucket: Save/View/Open of CDX forrmat related issues Priority: High Severity: High High level visibility to the user User`s Feedback labels Feb 3, 2025
@Zhirnoff Zhirnoff changed the title Ketcher structure copy-paste issue with ChemDraw Ketcher structure copy-paste in KET instead of MOL Feb 4, 2025
@Zhirnoff Zhirnoff removed the CDX-ChemDraw Bucket: Save/View/Open of CDX forrmat related issues label Feb 4, 2025
svvald added a commit that referenced this issue Feb 5, 2025
@svvald
#6411 – Revert copying functionality changes
4233ada
@rrodionov91 rrodionov91 self-assigned this Feb 5, 2025
@rrodionov91 rrodionov91 linked a pull request Feb 5, 2025 that will close this issue
#6411 – Ketcher structure copy-paste in KET instead of MOL #6420
Merged
9 tasks
rrodionov91 pushed a commit that referenced this issue Feb 5, 2025
@svvald
#6411 – Revert copying functionality changes (#6420)
9596494
rrodionov91 pushed a commit that referenced this issue Feb 5, 2025
@svvald @rrodionov91
#6411 – Revert copying functionality changes (#6420)
1ee7b8f
rrodionov91 pushed a commit that referenced this issue Feb 5, 2025
@svvald @rrodionov91
#6411 – Revert copying functionality changes (#6420)
0528928
This was linked to pull requests Feb 5, 2025
Backmerge: #6411 – Ketcher structure copy-paste in KET instead of MOL #6424
Merged
Backmerge: #6411 – Ketcher structure copy-paste in KET instead of MOL #6426
Merged
@Zhirnoff
Copy link
Collaborator Author

Zhirnoff commented Feb 6, 2025

Tested. Bug fixed. Also confirmed by user who have ChemDraw version 22.

Image

Desktop:

  • OS: Windows 11
  • Browser Chrome
  • Version 131.0.6778.205 (Official Build) (64-bit)

Ketcher version
[Version 3.0.0-rc.7]
Indigo version
[Version 1.28.0-rc.3]

rrodionov91 added a commit that referenced this issue Feb 6, 2025
@rrodionov91 @svvald
#6411 – Revert copying functionality changes (#6420) (#6424)
5bf9f53 
Co-authored-by: Nikita Chistousov <[email protected]>
rrodionov91 added a commit that referenced this issue Feb 6, 2025
@rrodionov91 @svvald
Backmerge: #6411 – Ketcher structure copy-paste in KET instead of MOL (…
260e96c 
…#6426)

* #6411 – Revert copying functionality changes (#6420)

* - updated screenshot

* - updated test

---------

Co-authored-by: Nikita Chistousov <[email protected]>
@AlexeyGirin AlexeyGirin mentioned this issue Mar 4, 2025
Open
38 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rrodionov91 rrodionov91

Labels
Priority: High Severity: High High level visibility to the user User`s Feedback
Projects
None yet
Milestone
Ketcher 3.0.0-rc.7
2 participants
@rrodionov91 @Zhirnoff