fix: sidebar height lp#2021965 (#4946)

canonical · May 31, 2023 · a329df3 · a329df3
1 parent 0f27868
commit a329df3
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/src/app/App.tsx b/src/app/App.tsx
@@ -165,9 +165,7 @@ export const App = (): JSX.Element => {
         <main className="l-main">
           {isSideNavVisible ? (
             <div
-              className={classNames("l-main__nav", {
-                "is-open": isSideNavVisible,
-              })}
+              className={classNames("l-main__nav", `is-maas-${theme}--accent`)}
             >
               <SecondaryNavigation
                 isOpen={!!isSideNavVisible}

diff --git a/src/app/base/components/SecondaryNavigation/_index.scss b/src/app/base/components/SecondaryNavigation/_index.scss
@@ -17,7 +17,7 @@
 
     &,
     .p-side-navigation__drawer {
-      height: 100%;
+      min-height: 100%;
     }
     .p-side-navigation__drawer {
       transform: none;

diff --git a/src/scss/_patterns_application.scss b/src/scss/_patterns_application.scss
@@ -10,6 +10,7 @@
       flex-shrink: 0;
       overflow-y: auto;
       height: 30%;
+      scrollbar-gutter: stable;
 
       @media (min-width: $breakpoint-small) {
         height: calc(100vh - 2.5rem);