Fixed mobile scrolling issues

fixes #12 - added background color to the mobile menu actions - disabled root scrolling when the mobile menu is opened which fixes the duplicate scrolling issue
TryGhost · Jan 3, 2024 · c060944 · c060944
1 parent d046267
commit c060944
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 4 deletions.
diff --git a/assets/built/screen.css b/assets/built/screen.css
diff --git a/assets/built/screen.css.map b/assets/built/screen.css.map
diff --git a/assets/built/source.js b/assets/built/source.js
diff --git a/assets/built/source.js.map b/assets/built/source.js.map
diff --git a/assets/css/screen.css b/assets/css/screen.css
@@ -515,6 +515,7 @@ button.gh-form-input {
     gap: 24px;
     align-items: center;
     justify-content: flex-end;
+    background-color: var(--background-color);
 }
 
 .gh-navigation-members {

diff --git a/assets/js/casper.js → assets/js/main.js b/assets/js/casper.js → assets/js/main.js
@@ -7,8 +7,10 @@
     burger.addEventListener('click', function () {
         if (!navigation.classList.contains('is-open')) {
             navigation.classList.add('is-open');
+            document.documentElement.style.overflowY = 'hidden';
         } else {
             navigation.classList.remove('is-open');
+            document.documentElement.style.overflowY = null;
         }
     });
 })();