From 4442ef199072b7e581783975d9b9cb107301cea0 Mon Sep 17 00:00:00 2001 From: juanatsap Date: Fri, 7 Nov 2025 21:40:41 +0000 Subject: [PATCH] fix: prevent hamburger menu from appearing during scroll when closed - Only apply header-hidden class to navigation menu when it's open - Check for menu-open class before hiding/showing menu on scroll - Closed menu now stays hidden regardless of scroll direction --- templates/index.html | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/templates/index.html b/templates/index.html index bb7b46e..2aa0d49 100644 --- a/templates/index.html +++ b/templates/index.html @@ -409,22 +409,32 @@ const navMenu = document.querySelector('.navigation-menu'); const backToTopBtn = document.getElementById('back-to-top'); const currentScroll = window.pageYOffset || document.documentElement.scrollTop; + const isMenuOpen = navMenu.classList.contains('menu-open'); // Hide/show header based on scroll direction if (currentScroll > scrollThreshold) { if (currentScroll > lastScrollTop) { // Scrolling down - hide header actionBar.classList.add('header-hidden'); - navMenu.classList.add('header-hidden'); + // Only hide menu if it's open + if (isMenuOpen) { + navMenu.classList.add('header-hidden'); + } } else { // Scrolling up - show header actionBar.classList.remove('header-hidden'); - navMenu.classList.remove('header-hidden'); + // Only show menu if it's open + if (isMenuOpen) { + navMenu.classList.remove('header-hidden'); + } } } else { // At top - always show header actionBar.classList.remove('header-hidden'); - navMenu.classList.remove('header-hidden'); + // Only affect menu if it's open + if (isMenuOpen) { + navMenu.classList.remove('header-hidden'); + } } // Show/hide back to top button