From 17c1ad06a04cb6eb3c7afeae86784a8e0ffb4955 Mon Sep 17 00:00:00 2001
From: MDLeom <2809763-curben@users.noreply.gitlab.com>
Date: Sat, 4 Apr 2020 06:53:32 +0100
Subject: [PATCH] feat(js): hide mobile menu when click outside

- https://htmldom.dev/detect-clicks-outside-of-an-element
---
 themes/chameleon/layout/_partial/header.ejs |  6 +++---
 themes/chameleon/source/css/chameleon.css   |  8 ++++----
 themes/chameleon/source/js/chameleon.js     | 20 ++++++++++++++++++--
 3 files changed, 25 insertions(+), 9 deletions(-)

diff --git a/themes/chameleon/layout/_partial/header.ejs b/themes/chameleon/layout/_partial/header.ejs
index 3ff5653..5737ee8 100644
--- a/themes/chameleon/layout/_partial/header.ejs
+++ b/themes/chameleon/layout/_partial/header.ejs
@@ -1,6 +1,6 @@
 <header id="header" class="header">
   <%/* Nav menu for desktop */%>
-  <nav id="main-nav" class="main-nav">
+  <nav class="main-nav">
     <% for (const i in theme.menu) { %>
       <a class="main-nav-link" href="<%- url_for(theme.menu[i]) %>"><%= i %></a>
     <% } %>
@@ -26,9 +26,9 @@
   <nav class="mobile-nav">
     <h1 class="site-title"><a href="<%- url_for(theme.menu['Home']) %>"><%= config.title %></a></h1>
     <ul class="mobile-nav-menu">
-    <label for="mobile-menu-toggle"><a class="menu-button no-underline">&#9776;</a></label>
+    <label for="mobile-menu-toggle"><a class="no-underline" id="menu-button">&#9776;</a></label>
     <input id="mobile-menu-toggle" type="checkbox"/>
-    <ul class="mobile-nav-link">
+    <ul id="mobile-nav-link">
       <div class="search-container">
         <form id="searchFormMob" method="post" action="https://duckduckgo.com/" target="_blank" rel="noopener external nofollow noreferrer">
         <input type="text" name="q" class="search-box" placeholder="Search..." title="Powered by DuckDuckGo">
diff --git a/themes/chameleon/source/css/chameleon.css b/themes/chameleon/source/css/chameleon.css
index 85858fb..76a1cb4 100644
--- a/themes/chameleon/source/css/chameleon.css
+++ b/themes/chameleon/source/css/chameleon.css
@@ -618,7 +618,7 @@ td {
 /* when it's not mobile */
 .mobile-nav,
 #mobile-menu-toggle,
-.mobile-nav-link {
+#mobile-nav-link {
   display: none;
 }
 
@@ -628,7 +628,7 @@ td {
 }
 
 /* overlap other elements */
-.mobile-nav-link {
+#mobile-nav-link {
   background: var(--main-bg-color);
   color: var(--main-font-color);
   border: 2px solid #999;
@@ -643,7 +643,7 @@ td {
   text-align: center;
 }
 
-.menu-button {
+#menu-button {
   font-size: 2em;
 }
 
@@ -764,7 +764,7 @@ svg#share:hover {
 
   /* display links when menu button is clicked */
   /* use grid to display each link in new line */
-  #mobile-menu-toggle:checked + .mobile-nav-link {
+  #mobile-menu-toggle:checked + #mobile-nav-link {
     display: grid;
   }
 }
diff --git a/themes/chameleon/source/js/chameleon.js b/themes/chameleon/source/js/chameleon.js
index 7271ea5..af42a1d 100644
--- a/themes/chameleon/source/js/chameleon.js
+++ b/themes/chameleon/source/js/chameleon.js
@@ -7,6 +7,22 @@ document.getElementById('search-click-mobile').addEventListener('click', () => {
   searchFormMob.submit()
 }, false)
 
+// Hide mobile menu when click outside of the menu
+document.addEventListener('click', (evt) => {
+  const mainNavDisplay = window.getComputedStyle(document.getElementsByClassName('main-nav')[0]).getPropertyValue('display')
+  const mobileNav = document.getElementById('mobile-nav-link')
+  const mobileToggle = document.getElementById('mobile-menu-toggle')
+  const isClickedOutside = !mobileNav.contains(evt.target)
+
+  // Exit if not in mobile view or menu button is clicked
+  // Menu button click triggers `menu-button` and `mobile-menu-toggle`
+  if (mainNavDisplay !== 'none' || evt.target.id === 'menu-button' || evt.target.id === 'mobile-menu-toggle') return
+
+  if (isClickedOutside) {
+    mobileToggle.checked = false
+  }
+}, false)
+
 // Web Share API
 // https://developer.mozilla.org/en-US/docs/Web/API/Navigator/share
 // Only available on supporting browsers and HTTPS
@@ -15,11 +31,11 @@ if (navigator.share && document.location.protocol === 'https:') {
     const query = (selector) => {
       return document.querySelector(selector)
     }
-  
+
     const title = query('meta[property="og:title"]') ? query('meta[property="og:title"]').content : ''
     const text = query('meta[property="og:description"]') ? query('meta[property="og:description"]').content : ''
     const url = query('link[rel="canonical"]') ? query('link[rel="canonical"]').href : document.location.href
-  
+
     await navigator.share({ title, text, url })
   }, false)
 }