#4941 - System shows Edit S-Group option for every bond of molecule i…

…f it has attachment point
epam · Jul 2, 2024 · 23a6f28 · 23a6f28
1 parent e42b47d
commit 23a6f28
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/...ges/ketcher-react/src/script/ui/views/components/ContextMenu/hooks/useBondSGroupAttach.ts b/...ges/ketcher-react/src/script/ui/views/components/ContextMenu/hooks/useBondSGroupAttach.ts
@@ -38,7 +38,11 @@ const useBondSGroupAttach = () => {
 
       const bond = struct.bonds.get(bondIds[0])!;
       const attachedSGroups = bond.b.getAttachedSGroups(struct.molecule);
-
+      const [sgGroupId] = attachedSGroups;
+      const sgroup = struct.sgroups.get(sgGroupId)?.item;
+      if (sgroup?.isSuperatomWithoutLabel) {
+        return false;
+      }
       return attachedSGroups.size > 0;
     },
     [getKetcherInstance],

diff --git a/packages/ketcher-react/src/script/ui/views/components/ContextMenu/hooks/useBondSGroupEdit.ts b/packages/ketcher-react/src/script/ui/views/components/ContextMenu/hooks/useBondSGroupEdit.ts
@@ -35,8 +35,15 @@ const useBondSGroupEdit = () => {
   );
 
   const hidden = useCallback(() => {
+    const editor = getKetcherInstance().editor as Editor;
+    const struct: ReStruct = editor.render.ctab;
+    const [sgGroupId] = sGroupsRef.current;
+    const sgroup = struct.sgroups.get(sgGroupId)?.item;
+    if (sgroup?.isSuperatomWithoutLabel) {
+      return true;
+    }
     return sGroupsRef.current.size === 0;
-  }, []);
+  }, [getKetcherInstance]);
 
   return [handler, disabled, hidden] as const;
 };