trafficlunar/blockmatic
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: use useRef() for oldTool instead of useState()

Browse source
This commit is contained in:
trafficlunar 2025-01-18 21:54:35 +00:00
parent dcf4cea0c4
commit 2037ad8722
1 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/components/canvas/Canvas.tsx
View file

@ -42,8 +42,8 @@ function Canvas() {
const [dragging, setDragging] = useState(false);
const [holdingAlt, setHoldingAlt] = useState(false);
const [oldTool, setOldTool] = useState<Tool>("hand");
const selectionBoxBoundsRef = useRef<BoundingBox>();
const oldToolRef = useRef<Tool>();
const visibleArea = useMemo(() => {
const blockSize = 16 * scale;
@ -302,7 +302,7 @@ function Canvas() {
switch (e.key) {
case " ": // Space
setDragging(true);
setOldTool(tool);
oldToolRef.current = tool;
setTool("hand");
setCssCursor("grabbing");
break;
@ -344,9 +344,10 @@ function Canvas() {
const onKeyUp = (e: KeyboardEvent) => {
switch (e.key) {
case " ": // Space
if (!oldToolRef.current) return;
setDragging(false);
setCssCursor("grab");
setTool(oldTool);
setTool(oldToolRef.current);
break;
case "Alt":
setHoldingAlt(false);