From 33760042d182ba6fdbe86ea86a3c3e5bbf870ac7 Mon Sep 17 00:00:00 2001 From: Zach Plata Date: Fri, 26 May 2023 14:22:27 -0500 Subject: [PATCH] fix recursive useEffect call --- src/hooks/useResizeCanvas.ts | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/hooks/useResizeCanvas.ts b/src/hooks/useResizeCanvas.ts index 60fa25b..2938e4c 100644 --- a/src/hooks/useResizeCanvas.ts +++ b/src/hooks/useResizeCanvas.ts @@ -62,6 +62,8 @@ export default function useResizeCanvas({ ); const currentDevicePixelRatio = useDevicePixelRatio(); + const { maxX, maxY } = artboardBounds ?? {}; + const getCanvasDimensions = useCallback(() => { const width = containerRef.current?.clientWidth ?? 0; const height = containerRef.current?.clientHeight ?? 0; @@ -73,7 +75,7 @@ export default function useResizeCanvas({ width, height, }; - }, [containerRef, fitCanvasToArtboardHeight, artboardBounds]); + }, [containerRef, fitCanvasToArtboardHeight, maxX, maxY]); useEffect(() => { if ( @@ -131,7 +133,7 @@ export default function useResizeCanvas({ if (onCanvasHasResized && (isFirstSizing || hasResized)) { onCanvasHasResized && onCanvasHasResized(); } - setIsFirstSizing(false); + isFirstSizing && setIsFirstSizing(false); }, [ canvasRef, containerRef,