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.



git-svn-id: https://develop.svn.wordpress.org/trunk@60302 602fd350-edb4-49c9-b593-d223f7449a82

Author: audrasjb

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

@@ -433,9 +433,13 @@
 			white-space: inherit;
 		}
 
+		&:not(:has(.sub-menu.expanded-true)) {
+			overflow-y: scroll;
+		}
+
 		&.expanded-true {
 
-			display: table;
+			display: block;
 			margin-top: 0;
 			opacity: 1;
 			padding-left: 0;

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

@@ -3271,8 +3271,12 @@ body.page .main-navigation {
   white-space: inherit;
 }
 
+.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;
   padding-right: 0;

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

@@ -3271,8 +3271,12 @@ body.page .main-navigation {
   white-space: inherit;
 }
 
+.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;
   padding-left: 0;