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

Connection of bond to atom inside s-group supergroup cause it connected incorrectly in terms of layout #4144

Open
AlexeyGirin opened this issue Feb 23, 2024 · 2 comments · Fixed by #4367
Open

Connection of bond to atom inside s-group supergroup cause it connected incorrectly in terms of layout #4144

AlexeyGirin opened this issue Feb 23, 2024 · 2 comments · Fixed by #4367
Assignees
@baranovdv
Labels
bug Micromolecules canvas Bucket: Micromolecules canvas related issues Priority: Medium
Milestone
Refined Backlog

Comments

@AlexeyGirin
Copy link
Collaborator

Describe the bug
Connection of bond to atom inside s-group supergroup cause it connected incorrectly in terms of layout

Steps to Reproduce

  1. Load from file: Connection of bond to atom inside s-group supergroup cause it connected incorrectly in terms of layout.zip
    image
  2. Connect bond and the ring as shown on the pic:
    image
  3. Contract abbreviation

Actual behavior
Abbreviation label looks not attached to the bond
image

Expected behavior
Abbreviation label is attached to the bond
image

Environment details (please complete the following information):
@AlexeyGirin AlexeyGirin added this to the Ketcher 2.21.0-rc.1 milestone Feb 26, 2024
baranovdv added a commit that referenced this issue Mar 28, 2024
@baranovdv
#4144 - Connection of bond to atom inside s-group supergroup cause it…
481d89c 
… connected incorrectly in terms of layout

- add findAttachmentPoints function
@baranovdv baranovdv linked a pull request Mar 28, 2024 that will close this issue
#4144 - Connection of bond to atom inside s-group supergroup cause it connected incorrectly in terms of layout #4367
Merged
9 tasks
NataliaLoginova added a commit that referenced this issue Apr 1, 2024
@NataliaLoginova
Merge pull request #4367 from epam/4144-connection-of-bond-to-atom-in…
8f23da6 
…side-s-group-supergroup-cause-it-connected-incorrectly-in-terms-of-layout

#4144 - Connection of bond to atom inside s-group supergroup cause it connected incorrectly in terms of layout
@epam epam deleted a comment from NataliaLoginova Apr 10, 2024
@AlexeyGirin
Copy link
Collaborator Author

Not fixed.
Ketcher Version 2.21.0-rc.1 Build at 2024-04-15; 20:00:15
Indigo Toolkit Version 1.20.0-rc.1.0-g8e8ffc3c3-wasm32-wasm-clang-12.0.0

image

@AlexeyGirin AlexeyGirin reopened this Apr 16, 2024
@AlexeyGirin AlexeyGirin added Micromolecules canvas Bucket: Micromolecules canvas related issues and removed query features phase II labels Apr 20, 2024
@AlexeyGirin
Copy link
Collaborator Author

Moved to Refined Backlog as per process

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

@baranovdv baranovdv

Labels
bug Micromolecules canvas Bucket: Micromolecules canvas related issues Priority: Medium
Projects
None yet
Milestone
Refined Backlog
4 participants
@NataliaLoginova @rrodionov91 @AlexeyGirin @baranovdv