fix: remove css :has in mobile sidebar to support all browsers

acid-info · Jun 7, 2023 · 2d5179c · 2d5179c
1 parent 038411e
commit 2d5179c
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 9 deletions.
diff --git a/packages/logos-docusaurus-theme/src/client/css/custom.scss b/packages/logos-docusaurus-theme/src/client/css/custom.scss
@@ -941,18 +941,17 @@ a[class^='sidebarLogo_'] {
     padding-bottom: 0px !important;
   }
 
-  .navbar-sidebar:has(.navbar-sidebar__items--show-secondary)
+  .navbar-sidebar:not(.navbar-sidebar--show-secondary) {
     .navbar-sidebar__brand {
-    button:first-of-type {
-      display: none;
-    }
+      button:first-of-type {
+        display: none;
+      }
 
-    button.lsd-icon-button {
-      margin-left: auto;
+      button.lsd-icon-button {
+        margin-left: auto;
+      }
     }
-  }
 
-  .navbar-sidebar:not(:has(.navbar-sidebar__items--show-secondary)) {
     .menu {
       padding-inline: 0;
     }

diff --git a/packages/logos-docusaurus-theme/src/client/theme/Navbar/MobileSidebar/Layout/index.tsx b/packages/logos-docusaurus-theme/src/client/theme/Navbar/MobileSidebar/Layout/index.tsx
@@ -11,7 +11,12 @@ export default function NavbarMobileSidebarLayout({
   const { shown: secondaryMenuShown } = useNavbarSecondaryMenu()
 
   return (
-    <div className="navbar-sidebar">
+    <div
+      className={clsx(
+        'navbar-sidebar',
+        secondaryMenuShown && 'navbar-sidebar--show-secondary',
+      )}
+    >
       {header}
       <div
         className={clsx('navbar-sidebar__items', {