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.

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

Bond spacing is incorrect after saving in PNG (svg) #5628

Closed
AnnaKosol opened this issue Sep 30, 2024 · 2 comments · Fixed by #5722
Closed

Bond spacing is incorrect after saving in PNG (svg) #5628

AnnaKosol opened this issue Sep 30, 2024 · 2 comments · Fixed by #5722

Comments

@AnnaKosol
Copy link
Contributor

Steps to Reproduce

  1. add benzene ring
  2. set bond length 50 px in the Settings on the Bond section
  3. set bond spacing 50%
  4. click on Apply button
  5. save to the png
  6. open saving file

Expected behavior
the picture should be as on the Canvas and saving preview
Screenshot 2024-09-30 at 17 21 59

Actual behavior
png low

Environment details:

Issues found while testing epam/Indigo#2175

@AliaksandrDziarkach
Copy link
Collaborator

Ketcher send ACS options to generate preview but not send when "Save" pressed:
For preview:
image
After "Save" pressed:
image

@AnnaKosol AnnaKosol changed the title Preview: bond spacing is incorrect after saving in PNG (svg) Bond spacing is incorrect after saving in PNG (svg) Oct 7, 2024
@mnmsvlw mnmsvlw assigned mnmsvlw and unassigned mnmsvlw Oct 8, 2024
@rrodionov91 rrodionov91 reopened this Oct 9, 2024
@NataliaLoginova NataliaLoginova linked a pull request Oct 11, 2024 that will close this issue
9 tasks
@AlexeyGirin
Copy link
Collaborator

Verified.
image

  • Ketcher Version 2.26.0-rc.2 Build at 2024-10-14; 19:59:03
  • Indigo Toolkit Version 1.25.0-rc.2.0-g867ae250b-wasm32-wasm-clang-19.0.0
  • Chrome Version 129.0.6668.101 (Official Build) (64-bit)
  • Win10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants