From 78af36bb92694f36966ee65faaa29bb9091c543a Mon Sep 17 00:00:00 2001
From: Wille Marcel <wille.yyz@gmail.com>
Date: Mon, 2 Sep 2024 17:10:56 -0300
Subject: [PATCH] DIsmiss mobile Nav after clicking on menu item

---
 src/components/header.js        | 21 +++++++++------------
 src/components/layout/layout.js |  7 +------
 src/components/nav.js           | 17 ++++++++++-------
 src/components/sticky-banner.js | 18 ++++++++++--------
 4 files changed, 30 insertions(+), 33 deletions(-)

diff --git a/src/components/header.js b/src/components/header.js
index ac3b18d4..65645948 100644
--- a/src/components/header.js
+++ b/src/components/header.js
@@ -5,10 +5,11 @@ import { Link } from "gatsby"
 
 import { colors, layout, breakpoints } from "../theme"
 import { CaseiLogoIcon, CloseIcon, HamburgerIcon } from "../icons"
-import { NEGATIVE } from "../utils/constants"
+import { NEGATIVE, POSITIVE } from "../utils/constants"
 import StickyBanner from "./sticky-banner"
 import Button from "./button"
 import NasaLogoIcon from "../icons/nasa-logo"
+import NavList from "./nav"
 
 const reveal = keyframes`
 0% {
@@ -173,18 +174,11 @@ const PageNavGlobalStyle = createGlobalStyle`
   }
 `
 
-const Header = ({ shortname, children, mode }) => {
-  const offsetCalculator = (scrollDirection, _, currentScroll) => {
-    if (scrollDirection === "scroll-down" && currentScroll > 250) {
-      return `-${document.getElementById("main-header").clientHeight}px`
-    } else {
-      return 0
-    }
-  }
-
+const Header = ({ shortname, mode }) => {
   const [navRevealed, setNavRevealed] = useState(false)
+
   return (
-    <StickyBanner offsetCalculator={offsetCalculator} navRevealed={navRevealed}>
+    <StickyBanner navRevealed={navRevealed}>
       <PageHeaderSelf id="main-header" mode={mode}>
         <PageHeaderInner>
           <PageHeadline>
@@ -225,7 +219,10 @@ const Header = ({ shortname, children, mode }) => {
             role="navigation"
             revealed={navRevealed}
           >
-            {children}
+            <NavList
+              mode={POSITIVE}
+              onLinkClick={() => setNavRevealed(false)}
+            />
           </PageNavWrapper>
         </PageHeaderInner>
       </PageHeaderSelf>
diff --git a/src/components/layout/layout.js b/src/components/layout/layout.js
index 08b57f1d..b50f8ee6 100644
--- a/src/components/layout/layout.js
+++ b/src/components/layout/layout.js
@@ -17,7 +17,6 @@ import "@mapbox/mapbox-gl-draw/dist/mapbox-gl-draw.css"
 import "../global.css"
 
 import Header from "../header"
-import Nav from "../nav"
 import Footer from "../footer"
 
 import { POSITIVE } from "../../utils/constants"
@@ -45,12 +44,8 @@ const Layout = ({ children }) => {
 
   return (
     <Container id="top" data-cy="page">
-      <Header shortname={data.site.siteMetadata.shortname} mode={POSITIVE}>
-        <Nav mode={POSITIVE} />
-      </Header>
-
+      <Header shortname={data.site.siteMetadata.shortname} mode={POSITIVE} />
       <main>{children}</main>
-
       <Footer shortname={data.site.siteMetadata.shortname} />
     </Container>
   )
diff --git a/src/components/nav.js b/src/components/nav.js
index 0b798b97..48a022dc 100644
--- a/src/components/nav.js
+++ b/src/components/nav.js
@@ -24,7 +24,7 @@ const GlobalMenu = styled.ul`
     flex-flow: row nowrap;
   }
 `
-const ListLink = ({ to, children, mode }) => (
+const ListLink = ({ to, children, mode, onClick }) => (
   <li
     css={`
       margin: 0;
@@ -47,6 +47,7 @@ const ListLink = ({ to, children, mode }) => (
         color: ${colors[mode].text};
       `}
       partiallyActive={true}
+      onClick={onClick}
     >
       {children}
     </Link>
@@ -71,24 +72,25 @@ ListLink.propTypes = {
   },
   children: PropTypes.string.isRequired,
   mode: PropTypes.oneOf([POSITIVE, NEGATIVE]),
+  onClick: PropTypes.func,
 }
 
-const NavList = ({ mode }) => {
+const NavList = ({ mode, onLinkClick }) => {
   return (
     <GlobalMenu>
-      <ListLink to="/explore/campaigns" mode={mode}>
+      <ListLink to="/explore/campaigns" mode={mode} onClick={onLinkClick}>
         Explore
       </ListLink>
-      <ListLink to="/glossary" mode={mode}>
+      <ListLink to="/glossary" mode={mode} onClick={onLinkClick}>
         Glossary
       </ListLink>
-      <ListLink to="/about" mode={mode}>
+      <ListLink to="/about" mode={mode} onClick={onLinkClick}>
         About
       </ListLink>
-      <ListLink to="/faq" mode={mode}>
+      <ListLink to="/faq" mode={mode} onClick={onLinkClick}>
         FAQS
       </ListLink>
-      <ListLink to="/contact" mode={mode}>
+      <ListLink to="/contact" mode={mode} onClick={onLinkClick}>
         Contact
       </ListLink>
     </GlobalMenu>
@@ -99,4 +101,5 @@ export default NavList
 
 NavList.propTypes = {
   mode: PropTypes.string.isRequired,
+  onLinkClick: PropTypes.func,
 }
diff --git a/src/components/sticky-banner.js b/src/components/sticky-banner.js
index ac4865c8..d878dec6 100644
--- a/src/components/sticky-banner.js
+++ b/src/components/sticky-banner.js
@@ -1,7 +1,7 @@
 import React, { useEffect, useState } from "react"
 import PropTypes from "prop-types"
 
-const StickyBanner = ({ children, offsetCalculator, navRevealed }) => {
+const StickyBanner = ({ children, navRevealed }) => {
   const scrollDown = "scroll-down"
   const scrollUp = "scroll-up"
   const [scrollDirection, setScrollDirection] = useState(null)
@@ -10,6 +10,14 @@ const StickyBanner = ({ children, offsetCalculator, navRevealed }) => {
   const [offset, setOffset] = useState(0)
   const node = React.createRef()
 
+  const offsetCalculator = (scrollDirection, _, currentScroll) => {
+    if (scrollDirection === "scroll-down" && currentScroll > 250) {
+      return `-${document.getElementById("main-header").clientHeight}px`
+    } else {
+      return 0
+    }
+  }
+
   useEffect(() => {
     if (startingPosition === null) {
       // set position on first load
@@ -41,12 +49,7 @@ const StickyBanner = ({ children, offsetCalculator, navRevealed }) => {
     }
     setLastScroll(currentScroll)
     setOffset(
-      offsetCalculator(
-        scrollDirection,
-        startingPosition,
-        currentScroll,
-        lastScroll
-      )
+      offsetCalculator(scrollDirection, startingPosition, currentScroll)
     )
   }
 
@@ -71,7 +74,6 @@ const StickyBanner = ({ children, offsetCalculator, navRevealed }) => {
 StickyBanner.propTypes = {
   children: PropTypes.element,
   hideAfter: PropTypes.number,
-  offsetCalculator: PropTypes.func,
   navRevealed: PropTypes.bool,
 }