Merge pull request #63 from quadratichq/62-bug-selecting-cells-moving…

…-offscreen 62 bug selecting cells moving offscreen
quadratichq · May 23, 2022 · 7018a7e · 7018a7e
2 parents eab768b + 54e8c12
commit 7018a7e
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 7 deletions.
diff --git a/docs/examples/monte_carlo_simulation.grid b/docs/examples/monte_carlo_simulation.grid
diff --git a/docs/examples/startup_portfolio.grid b/docs/examples/startup_portfolio.grid
diff --git a/src/core/gridGL/interaction/ViewportEventRegister.tsx b/src/core/gridGL/interaction/ViewportEventRegister.tsx
@@ -17,13 +17,15 @@ export const ViewportEventRegister = (props: { viewport: Viewport }) => {
 
   // When the cursor moves ensure it is visible
   useEffect(() => {
-    viewport.ensureVisible(
-      interactionState.cursorPosition.x * CELL_WIDTH,
-      interactionState.cursorPosition.y * CELL_HEIGHT - 40,
-      CELL_WIDTH,
-      CELL_HEIGHT * 4,
-      false
-    );
+    // When multiCursor is visible don't force the single cursor to be visible
+    if (!interactionState.showMultiCursor)
+      viewport.ensureVisible(
+        interactionState.cursorPosition.x * CELL_WIDTH,
+        interactionState.cursorPosition.y * CELL_HEIGHT - 40,
+        CELL_WIDTH,
+        CELL_HEIGHT * 4,
+        false
+      );
   }, [viewport, interactionState]);
 
   // register zooming event listener to set Atom state