epam / ketcher

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

The picture is incorrect when image resolution high and bond thickness changing #5610

Closed AnnaKosol closed 1 month ago

AnnaKosol commented 1 month ago

Steps to Reproduce

  1. add benzene ring to the Canvas
  2. set image resolution High in the Settings
  3. set bond thickness 2.6 pt in the Settings in the Bond section
  4. click on Apply button
  5. save to png (or svg)

Expected behavior

Снимок экрана 2024-09-30 в 13 07 03

Actual behavior

Снимок экрана 2024-09-30 в 13 07 28

Environment details:

Issues found while testing https://github.com/epam/Indigo/issues/2175

AliaksandrDziarkach commented 1 month ago

This is a ketcher issue: image Both render-bond-line-width and render-bond-thickness set. render-bond-thickness set bond with as 2.6pt~=22px render-bond-line-width is image so result bond with is 22px*2.6~=56px render-bond-line-width should be set to 1

AnnaKosol commented 1 month ago

Verified that in the request Covert there are not the old options

Screenshot 2024-10-08 at 11 35 44

Environment https://rc.test.lifescience.opensource.epam.com/KetcherDemoSA/index.html Ketcher Version 2.26.0-rc.1 Build at 2024-10-04; 18:45:04 Indigo Version 1.25.0-rc.1.0-g704673fcb-wasm32-wasm-clang-19.0.0 MacOs Chrome 129.0.6668.71 (Official Build) (arm64)