From 18808481fdbe665fe5e37cb07b80148efcc03dc2 Mon Sep 17 00:00:00 2001 From: Ashwin Temkar <83207402+ashwintemkar@users.noreply.github.com> Date: Wed, 11 Jun 2025 20:22:02 +0530 Subject: [PATCH] fix: set cursor to auto when not hovering a point on linear element (#9642) * fix: set cursor to auto when not hovering a point on linear element #9628 * Simplify hover test for cursor * Add back comment * Fix test for hit testing --------- Co-authored-by: Mark Tolmacs --- packages/element/tests/collision.test.tsx | 2 +- packages/excalidraw/components/App.tsx | 5 ++++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/packages/element/tests/collision.test.tsx b/packages/element/tests/collision.test.tsx index bcbf114f5e..72996bdb1f 100644 --- a/packages/element/tests/collision.test.tsx +++ b/packages/element/tests/collision.test.tsx @@ -28,7 +28,7 @@ describe("check rotated elements can be hit:", () => { //const p = [120, -211]; //const p = [0, 13]; const hit = hitElementItself({ - point: pointFrom(87, -68), + point: pointFrom(88, -68), element: window.h.elements[0], threshold: 10, elementsMap: window.h.scene.getNonDeletedElementsMap(), diff --git a/packages/excalidraw/components/App.tsx b/packages/excalidraw/components/App.tsx index cdf5f1c906..beca285e1c 100644 --- a/packages/excalidraw/components/App.tsx +++ b/packages/excalidraw/components/App.tsx @@ -6147,7 +6147,10 @@ class App extends React.Component { setCursor(this.interactiveCanvas, CURSOR_TYPE.AUTO); } else if (isOverScrollBar) { setCursor(this.interactiveCanvas, CURSOR_TYPE.AUTO); - } else if (this.state.selectedLinearElement) { + } else if ( + this.state.selectedLinearElement && + hitElement?.id === this.state.selectedLinearElement.elementId + ) { this.handleHoverSelectedLinearElement( this.state.selectedLinearElement, scenePointerX,