Merge pull request #30 from bernhardoj/update-container-height

Dynamically update container height
Expensify · Jul 5, 2024 · b5534c7 · b5534c7
2 parents 1abc4ed + 403121d
commit b5534c7
Showing 1 changed file with 13 additions and 2 deletions.
diff --git a/src/PDFPreviewer.tsx b/src/PDFPreviewer.tsx
@@ -231,8 +231,19 @@ function PDFPreviewer({
     }, [isPasswordInvalid, attemptPDFLoad, setIsPasswordInvalid, renderPasswordForm]);
 
     useLayoutEffect(() => {
-        setContainerWidth(containerRef.current?.clientWidth ?? 0);
-        setContainerHeight(containerRef.current?.clientHeight ?? 0);
+        if (!containerRef.current) {
+            return undefined;
+        }
+        const resizeObserver = new ResizeObserver(() => {
+            if (!containerRef.current) {
+                return;
+            }
+            setContainerWidth(containerRef.current.clientWidth);
+            setContainerHeight(containerRef.current.clientHeight);
+        });
+        resizeObserver.observe(containerRef.current);
+
+        return () => resizeObserver.disconnect();
     }, []);
 
     return (