cleanup some state and functions

src/components/Footer/index.jsx

@@ -33,7 +33,7 @@ const socials = [
 ];
 
 export default function Footer() {
-    const toggleHome = () => {
+    const scrollToHome = () => {
         scroll.scrollToTop();
     };
 
@@ -43,7 +43,7 @@ export default function Footer() {
                 <FooterWrap>
                     <SocialMedia>
                         <SocialMediaWrap>
-                            <SocialLogo to="/udlbook" onClick={toggleHome}>
+                            <SocialLogo to="/udlbook" onClick={scrollToHome}>
                                 Understanding Deep Learning
                             </SocialLogo>
                             <WebsiteRights>

src/components/Navbar/index.jsx

@@ -17,11 +17,7 @@ export default function Navbar({ toggle }) {
 
     useEffect(() => {
         const changeNav = () => {
-            if (window.scrollY >= 80) {
+            setScrollNav(window.scrollY >= 80);
-                setScrollNav(true);
-            } else {
-                setScrollNav(false);
-            }
         };
 
         window.addEventListener("scroll", changeNav);
@@ -31,7 +27,7 @@ export default function Navbar({ toggle }) {
         };
     }, []);
 
-    const toggleHome = () => {
+    const scrollToHome = () => {
         scroll.scrollToTop();
     };
 
@@ -40,7 +36,7 @@ export default function Navbar({ toggle }) {
             <IconContext.Provider value={{ color: "#fff" }}>
                 <Nav scrollNav={scrollNav}>
                     <NavbarContainer>
-                        <NavLogo to="/udlbook/" onClick={toggleHome}>
+                        <NavLogo to="/udlbook/" onClick={scrollToHome}>
                             <h1> Understanding Deep Learning </h1>
                         </NavLogo>
                         <MobileIcon onClick={toggle}>

src/pages/index.jsx

@@ -12,7 +12,7 @@ export default function Index() {
     const [isOpen, setIsOpen] = useState(false);
 
     const toggle = () => {
-        setIsOpen(!isOpen);
+        setIsOpen((p) => !p);
     };
 
     return (