From 9e4aa1aea0c7cca4fca8619bc07b98c260fce6ce Mon Sep 17 00:00:00 2001 From: TheFlow Date: Mon, 9 Feb 2026 18:11:22 +1300 Subject: [PATCH] =?UTF-8?q?refactor:=20Clean=20up=20Research=20dropdown=20?= =?UTF-8?q?=E2=80=94=20navigational=20items=20only?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Remove individual paper titles from the Research dropdown. Replace with three navigational items: Publications (opens research papers modal), For Researchers, Research Timeline. Papers belong on pages, not in navigation menus. Co-Authored-By: Claude Opus 4.6 --- public/js/components/navbar.js | 77 +++++++++------------------------- 1 file changed, 19 insertions(+), 58 deletions(-) diff --git a/public/js/components/navbar.js b/public/js/components/navbar.js index b000d342..465bbc82 100644 --- a/public/js/components/navbar.js +++ b/public/js/components/navbar.js @@ -47,57 +47,20 @@ class TractatusNavbar { - @@ -217,9 +180,7 @@ class TractatusNavbar { @@ -430,10 +391,10 @@ class TractatusNavbar { mobileMenuBackdrop.addEventListener('click', toggleMobileMenu); } - // Close mobile menu on navigation - const mobileLinks = document.querySelectorAll('#mobile-menu a'); - mobileLinks.forEach(link => { - link.addEventListener('click', () => { + // Close mobile menu on navigation (links and modal triggers) + const mobileClickables = document.querySelectorAll('#mobile-menu a, #mobile-menu [data-research-papers-trigger]'); + mobileClickables.forEach(el => { + el.addEventListener('click', () => { if (this.mobileMenuOpen) { toggleMobileMenu(); }