Changeset 60302

Timestamp:

06/12/2025 12:40:38 PM (5 months ago)

Author:

audrasjb

Message:

Twenty Nineteen: Make mobile nav menu scrollable.

This changeset fixes an issue where Twenty Nineteen's primary nav menu panel wasn’t scrollable when the primary menu contains a lot of items, preventing to scroll down to see all menu items.

Props kjellr, karmatosed, lakshyajeet, coralietixeront, Spaceshipone, maxpertici, beryldlg.
Fixes #45902.

Location:

trunk/src/wp-content/themes/twentynineteen

Files:

• sass/navigation/_menu-main-navigation.scss (modified) (1 diff)
• style-rtl.css (modified) (1 diff)
• style.css (modified) (1 diff)

trunk/src/wp-content/themes/twentynineteen/sass/navigation/_menu-main-navigation.scss

@@ -434 +434 @@
         }
 
+        &:not(:has(.sub-menu.expanded-true)) {
+            overflow-y: scroll;
+        }
+
         &.expanded-true {
 
-            display: table;
+            display: block;
             margin-top: 0;
             opacity: 1;

trunk/src/wp-content/themes/twentynineteen/style-rtl.css

@@ -3272 +3272 @@
 }
 
+.main-navigation .main-menu .menu-item-has-children.off-canvas .sub-menu:not(:has(.sub-menu.expanded-true)) {
+  overflow-y: scroll;
+}
+
 .main-navigation .main-menu .menu-item-has-children.off-canvas .sub-menu.expanded-true {
-  display: table;
+  display: block;
   margin-top: 0;
   opacity: 1;

trunk/src/wp-content/themes/twentynineteen/style.css

@@ -3272 +3272 @@
 }
 
+.main-navigation .main-menu .menu-item-has-children.off-canvas .sub-menu:not(:has(.sub-menu.expanded-true)) {
+  overflow-y: scroll;
+}
+
 .main-navigation .main-menu .menu-item-has-children.off-canvas .sub-menu.expanded-true {
-  display: table;
+  display: block;
   margin-top: 0;
   opacity: 1;