#1887 Add s-group data for atom or bond without selection (#2080)

epam · Jan 11, 2023 · 57ec567 · 57ec567
1 parent 248f71d
commit 57ec567
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/packages/ketcher-react/src/script/editor/tool/sgroup.ts b/packages/ketcher-react/src/script/editor/tool/sgroup.ts
@@ -442,8 +442,10 @@ class SGroupTool {
     }
 
     // TODO: handle click on an existing group?
-    if (id !== null || (selection && selection.atoms))
+    if (id !== null || (selection && selection.atoms)) {
+      this.editor.selection(selection)
       SGroupTool.sgroupDialog(this.editor, id, this.type)
+    }
   }
 
   cancel() {
@@ -461,11 +463,6 @@ class SGroupTool {
     const type = sg ? sg.type : defaultType
     const eventName = type === 'DAT' ? 'sdataEdit' : 'sgroupEdit'
 
-    if (!selection.atoms && !selection.bonds && !sg) {
-      console.info('There is no selection or sgroup')
-      return
-    }
-
     let attrs
     if (sg) {
       attrs = sg.getAttrs()