From afca9cb27e721e064eb2c1d5c01c2e19439d2dee Mon Sep 17 00:00:00 2001
From: Rob Leslie <rob@mars.org>
Date: Tue, 4 Apr 2023 15:46:04 -0700
Subject: [PATCH] Fix problem showing dropdown after window resize

When the browser window is resized, a new click event listener is added
to the window but the old one is not removed. The old one ends up
preventing the newly-made dropdown from opening.

Fix this by remembering the click event listener function added to the
window, and remove it before making the new dropdown after a window
resize.
---
 assets/js/dropdown.js | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/assets/js/dropdown.js b/assets/js/dropdown.js
index 68ae516..aedd8b0 100644
--- a/assets/js/dropdown.js
+++ b/assets/js/dropdown.js
@@ -16,6 +16,7 @@
         });
     }
 
+    var windowClickListener;
     const makeDropdown = function () {
         if (mediaQuery.matches) return;
         const submenuItems = [];
@@ -62,11 +63,12 @@
             document.body.classList.toggle('is-dropdown-open');
         });
 
-        window.addEventListener('click', function (e) {
+        windowClickListener = function (e) {
             if (!toggle.contains(e.target) && document.body.classList.contains('is-dropdown-open')) {
                 document.body.classList.remove('is-dropdown-open');
             }
-        });
+        };
+        window.addEventListener('click', windowClickListener);
     }
 
     imagesLoaded(head, function () {
@@ -75,6 +77,7 @@
 
     window.addEventListener('resize', function () {
         setTimeout(function () {
+            window.removeEventListener('click', windowClickListener);
             nav.innerHTML = navHTML;
             makeDropdown();
         }, 1);