Backmerge: #4154 - Cannot select bonds when we add molecule with addF…

…ragment, setMolecule or Paste from clipboard

ketcher-autotests/tests/utils/files/readFile.ts

@@ -84,8 +84,6 @@ export async function openFileAndAddToCanvas(
 export async function openFileAndAddToCanvasAsNewProject(
   filename: string,
   page: Page,
-  xOffsetFromCenter?: number,
-  yOffsetFromCenter?: number,
 ) {
   await selectTopPanelButton(TopPanelButton.Open, page);
   await openFile(filename, page);
@@ -95,15 +93,6 @@ export async function openFileAndAddToCanvasAsNewProject(
   await waitForLoad(page, async () => {
     await pressButton(page, 'Open as New Project');
   });
-
-  if (
-    typeof xOffsetFromCenter === 'number' &&
-    typeof yOffsetFromCenter === 'number'
-  ) {
-    await clickOnTheCanvas(page, xOffsetFromCenter, yOffsetFromCenter);
-  } else {
-    await clickInTheMiddleOfTheScreen(page);
-  }
 }
 
 export async function filteredFile(

packages/ketcher-core/src/application/editor/operations/CanvasLoad.ts

@@ -33,7 +33,7 @@ export class CanvasLoad extends BaseOperation {
     const oldStruct = restruct.molecule;
     restruct.clearVisels(); // TODO: What is it?
     if (this.data.struct) {
-      restruct.render.setMolecule(this.data.struct);
+      restruct.render.setMolecule(this.data.struct, true);
     }
 
     this.data.struct = oldStruct;

packages/ketcher-core/src/application/render/raphaelRender.ts

@@ -191,11 +191,19 @@ export class Render {
     this.scrollbar.update();
   }
 
-  setMolecule(struct: Struct) {
+  setMolecule(struct: Struct, forceUpdateWithTimeout = false) {
     this.paper.clear();
     this.ctab = new ReStruct(struct, this);
     this.options.offset = new Vec2();
-    this.update(false);
+    // need to use force update with timeout to have ability select bonds in case of usage:
+    // addFragment, setMolecule or "Paste from clipboard" with "Open as New Project" button
+    if (forceUpdateWithTimeout) {
+      setTimeout(() => {
+        this.update(true);
+      }, 0);
+    } else {
+      this.update(false);
+    }
   }
 
   update(force = false, viewSz: Vec2 | null = null) {