From ade77b7ce67a06f3056d7de8a2d3715e80ce671d Mon Sep 17 00:00:00 2001 From: trafficlunar Date: Wed, 22 Jan 2025 20:24:40 +0000 Subject: [PATCH] fix: change cursor to `move` when using move tool --- src/components/canvas/Canvas.tsx | 1 + src/context/Tool.tsx | 17 ++++++----------- 2 files changed, 7 insertions(+), 11 deletions(-) diff --git a/src/components/canvas/Canvas.tsx b/src/components/canvas/Canvas.tsx index bc95716..fb4268b 100644 --- a/src/components/canvas/Canvas.tsx +++ b/src/components/canvas/Canvas.tsx @@ -82,6 +82,7 @@ function Canvas() { const updateCssCursor = useCallback(() => { const cursorMapping: Partial> = { hand: dragging ? "grab" : "grabbing", + move: "move", zoom: holdingAltRef.current ? "zoom-out" : "zoom-in", }; diff --git a/src/context/Tool.tsx b/src/context/Tool.tsx index 8f3e2f7..e151a87 100644 --- a/src/context/Tool.tsx +++ b/src/context/Tool.tsx @@ -27,18 +27,13 @@ export const ToolProvider = ({ children }: Props) => { const [cssCursor, setCssCursor] = useState("crosshair"); useEffect(() => { - switch (tool) { - case "hand": - setCssCursor("grab"); - break; - case "zoom": - setCssCursor("zoom-in"); - break; + const cursorMapping: Partial> = { + hand: "grab", + move: "move", + zoom: "zoom-in", + }; - default: - setCssCursor("crosshair"); - break; - } + setCssCursor(cursorMapping[tool] || "crosshair"); }, [tool]); return (