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

The bottom horizontal bonds does not shift if there is an overlap #4995

Open
AlexeyGirin opened this issue Jul 4, 2024 · 1 comment · Fixed by #5028 or #5037
Open

The bottom horizontal bonds does not shift if there is an overlap #4995

AlexeyGirin opened this issue Jul 4, 2024 · 1 comment · Fixed by #5028 or #5037
Assignees
@rrodionov91
Labels
Priority: Low Snake mode Bucket: Snake mode related issues
Milestone
Refined Backlog

Comments

@AlexeyGirin
Copy link
Collaborator

Steps to Reproduce

  1. Go to Macromolecules mode - Snake mode
  2. Load from file: The lower horizontal part of bond does not shift if there is an overlap.zip

Actual behavior
The bottom horizontal bonds does not shift if there is an overlap
image

Expected behavior
Bottom horizontal bonds should not overlap each other

Versions

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

Issue found while testing - #3532
@AlexeyGirin AlexeyGirin added bug Priority: Low Snake mode Bucket: Snake mode related issues labels Jul 4, 2024
@AlexeyGirin AlexeyGirin added this to the Ketcher 2.23.0-rc.2 milestone Jul 4, 2024
rrodionov91 added a commit that referenced this issue Jul 7, 2024
@rrodionov91
#4997 - The second of three horizontal bond overlaps backbone connect…
8d7d36a 
…ion line

- changed order of rendering bonds(side over backbone)

#4995 - The bottom horizontal bonds does not shift if there is an overlap

- applied offsets for bottom part of horizontal bonds
@rrodionov91 rrodionov91 self-assigned this Jul 7, 2024
@rrodionov91 rrodionov91 mentioned this issue Jul 7, 2024
Merged
9 tasks
@rrodionov91 rrodionov91 linked a pull request Jul 7, 2024 that will close this issue
#4997 - The second of three horizontal bond overlaps backbone connection line #5028
Merged
9 tasks
rrodionov91 added a commit that referenced this issue Jul 8, 2024
@rrodionov91
#4997 - The second of three horizontal bond overlaps backbone connect…
ed5ad1f 
…ion line (#5028)

#4997 - The second of three horizontal bond overlaps backbone connection line

- changed order of rendering bonds(side over backbone)

#4995 - The bottom horizontal bonds does not shift if there is an overlap

- applied offsets for bottom part of horizontal bonds
- updated tests/screenshots
- fixed bond color in flex mode
rrodionov91 added a commit that referenced this issue Jul 8, 2024
@rrodionov91
#4997 - The second of three horizontal bond overlaps backbone connect…
79d60fa 
…ion line

- changed order of rendering bonds(side over backbone)

#4995 - The bottom horizontal bonds does not shift if there is an overlap

- applied offsets for bottom part of horizontal bonds
@rrodionov91 rrodionov91 mentioned this issue Jul 8, 2024
Merged
9 tasks
rrodionov91 added a commit that referenced this issue Jul 8, 2024
@rrodionov91
Backmerge: #4997 - The second of three horizontal bond overlaps backb…
677beee 
…one connection line (#5037)

* #4997 - The second of three horizontal bond overlaps backbone connection line

- changed order of rendering bonds(side over backbone)

#4995 - The bottom horizontal bonds does not shift if there is an overlap

- applied offsets for bottom part of horizontal bonds
- updated screenshots
@AnnaKosol
Copy link
Contributor

The horizontal bottom overlaps the other one
Image
or in another case there is an overlap
Image
Versions:
Ketcher Version 2.23.0-rc.2 Build at 2024-07-10; 19:39:22
Indigo Version 1.22.0-rc.2.0-ga21fb5d8f-wasm32-wasm-clang-19.0.0
Chrome Версия 126.0.6478.127 (Официальная сборка), (arm64)
MacOs Sonoma 14.5

@AnnaKosol AnnaKosol reopened this Jul 11, 2024
@AlexeyGirin AlexeyGirin removed the bug label Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rrodionov91 rrodionov91

Labels
Priority: Low Snake mode Bucket: Snake mode related issues
Projects
None yet
Milestone
Refined Backlog
3 participants
@rrodionov91 @AlexeyGirin @AnnaKosol