Merge pull request #20 from outis1one/claude/automate-sam-download-mzpRs

Fix remaining onZoomChange references to use ref pattern
2026-01-26 00:24:36 -05:00
parent ece6e990d8 9cd8a5badb
commit bb859105ff
1 changed files with 9 additions and 3 deletions
@@ -659,7 +659,9 @@ const ImageCanvas = forwardRef(({
    if (newZoom > 10) newZoom = 10;
    canvas.setZoom(newZoom);
    setCurrentZoom(newZoom);
-    onZoomChange?.(newZoom);
+    if (onZoomChangeRef.current) {
+      onZoomChangeRef.current(newZoom);
+    }
  };

  const handleZoomOut = () => {
@@ -669,7 +671,9 @@ const ImageCanvas = forwardRef(({
    if (newZoom < 0.1) newZoom = 0.1;
    canvas.setZoom(newZoom);
    setCurrentZoom(newZoom);
-    onZoomChange?.(newZoom);
+    if (onZoomChangeRef.current) {
+      onZoomChangeRef.current(newZoom);
+    }
  };

  const handleZoomReset = () => {
@@ -678,7 +682,9 @@ const ImageCanvas = forwardRef(({
    canvas.setZoom(1);
    canvas.setViewportTransform([1, 0, 0, 1, 0, 0]);
    setCurrentZoom(1);
-    onZoomChange?.(1);
+    if (onZoomChangeRef.current) {
+      onZoomChangeRef.current(1);
+    }
  };

  return (