fix: only update position, when formatting toolbar is shown

dytab · Sep 17, 2024 · b5a6c16 · b5a6c16
1 parent c836292
commit b5a6c16
Showing 1 changed file with 11 additions and 9 deletions.
diff --git a/...note/src/lib/components/formatting-toolbar/bna-formatting-toolbar-controller.component.ts b/...note/src/lib/components/formatting-toolbar/bna-formatting-toolbar-controller.component.ts
@@ -50,15 +50,17 @@ export class BnaFormattingToolbarControllerComponent {
       cleanup();
       //TODO: remove auto update
       //had the problem that the first set position was not good
-      cleanup = autoUpdate(
-        getVirtualElement(formattingToolbar.referencePos),
-        this.elRef.nativeElement,
-        async () => {
-          await this.updateFormattingToolbarPosition(
-            formattingToolbar.referencePos,
-          );
-        },
-      );
+      if(formattingToolbar.show){
+        cleanup = autoUpdate(
+          getVirtualElement(formattingToolbar.referencePos),
+          this.elRef.nativeElement,
+          async () => {
+            await this.updateFormattingToolbarPosition(
+              formattingToolbar.referencePos,
+            );
+          },
+        );
+      }
     });
   }