Fix max-height constraint on mobile menu (#3491)

xh · Oct 10, 2023 · bc6ff3f · bc6ff3f
1 parent f9b4570
commit bc6ff3f
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/mobile/cmp/menu/impl/Menu.scss b/mobile/cmp/menu/impl/Menu.scss
@@ -2,7 +2,13 @@
   position: fixed;
   margin: 0 !important;
   z-index: 21;
-  max-height: 90vh;
+  /**
+   * Use `svh` units target the *smallest* viewport height, which is preferred to using `vh` on mobile.
+   * Including `px` units as a fallback for older browsers that may not support `vh`/`svh` units.
+   * https://css-tricks.com/the-large-small-and-dynamic-viewports/
+   */
+  max-height: 500px;
+  max-height: 80svh;
 
   &__title {
     padding: var(--xh-pad-half-px);