fix: move window into useEffect

calblueprint · May 26, 2024 · 444da2b · 444da2b
1 parent 8e0ff6d
commit 444da2b
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 23 deletions.
diff --git a/src/app/testing/page.tsx b/src/app/testing/page.tsx
diff --git a/src/components/userComponents/Footer/Footer.tsx b/src/components/userComponents/Footer/Footer.tsx
@@ -344,12 +344,14 @@ export default function Footer() {
   const [inputValueEmail, setEmailValue] = useState('');
   const [showError, setShowError] = useState(false);
   const [errorMsg, setErrorMsg] = useState('');
-
-  const subbed = () =>
-    Boolean(JSON.parse(window.sessionStorage.getItem('subscribed') || 'false'));
-  const [subscribed, setSubscribed] = useState(subbed);
+  const [subscribed, setSubscribed] = useState(false);
 
   useEffect(() => {
+    const subbed = () =>
+      Boolean(
+        JSON.parse(window.sessionStorage.getItem('subscribed') || 'false'),
+      );
+    setSubscribed(subbed);
     window.sessionStorage.setItem('subscribed', subscribed.toString());
   }, [subscribed]);
 

diff --git a/src/context/WindowWidthContext/WindowWidthContext.tsx b/src/context/WindowWidthContext/WindowWidthContext.tsx
@@ -6,6 +6,7 @@ import React, {
   useEffect,
   useContext,
   useMemo,
+  useCallback,
 } from 'react';
 
 interface WindowWidthContextProps {
@@ -29,15 +30,18 @@ export function WindowWidthProvider({
 }: {
   children: React.ReactNode;
 }) {
-  const [isWeb, setIsWeb] = useState(window.innerWidth >= 1024);
+  const [isWeb, setIsWeb] = useState(false);
+  const handleResize = useCallback(() => {
+    setIsWeb(window.innerWidth >= 1024);
+  }, []);
 
   useEffect(() => {
-    const handleResize = () => setIsWeb(window.innerWidth >= 1024);
+    handleResize();
     window.addEventListener('resize', handleResize);
     return () => {
       window.removeEventListener('resize', handleResize);
     };
-  }, []);
+  }, [handleResize]);
 
   const windowWidthValue = useMemo(() => ({ isWeb }), [isWeb]);