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

System shows Edit S-Group option for every bond of molecule if it has attachment point #4941

Closed
AlexeyGirin opened this issue Jun 29, 2024 · 0 comments · Fixed by #4964
Closed

System shows Edit S-Group option for every bond of molecule if it has attachment point #4941

AlexeyGirin opened this issue Jun 29, 2024 · 0 comments · Fixed by #4964
Assignees
@Guch1g0v
Labels
bug Macromolecules Priority: Medium
Milestone
Refined Backlog

Comments

@AlexeyGirin
Copy link
Collaborator

Steps to Reproduce

  1. Load from file: Unable to change atom to another if molecule has attachment point.zip
    OR manually create following configuration on the canvas:
    image
  2. Right-click on any bond of molecule

Actual behavior
System shows Edit S-Group option for every bond of molecule
image

Expected behavior
No Edit S-Group option in the context menu for every bond of molecule
image

Versions

  • Ketcher Version 2.22.0-rc.2 Build at 2024-06-28; 05:30:15
  • Indigo Toolkit Version 1.22.0-dev.2.1-gd5637f75-wasm32-wasm-clang-12.0.0
  • Windows 10 Pro
  • Chrome Version 126.0.6478.127 (Official Build) (64-bit)

Issue found while testing #4530
@AlexeyGirin AlexeyGirin added this to the Ketcher 2.23.0-rc.1 milestone Jun 29, 2024
@Guch1g0v Guch1g0v self-assigned this Jun 30, 2024
Guch1g0v added a commit that referenced this issue Jul 2, 2024
@Guch1g0v
#4941 - System shows Edit S-Group option for every bond of molecule i…
23a6f28 
…f it has attachment point
@Guch1g0v Guch1g0v linked a pull request Jul 2, 2024 that will close this issue
#4941 - System shows Edit S-Group option for every bond of molecule if it has attachment point #4964
Merged
9 tasks
NataliaLoginova pushed a commit that referenced this issue Oct 21, 2024
@Guch1g0v
#4941 - System shows Edit S-Group option for every bond of molecule i…
0ede499 
…f it has attachment point (#4964)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Guch1g0v Guch1g0v

Labels
bug Macromolecules Priority: Medium
Projects
None yet
Milestone
Refined Backlog
3 participants
@rrodionov91 @AlexeyGirin @Guch1g0v