fix(ld-sidenav): collapse fully if it has no primary mode nav items

emdgroup-liquid · Feb 14, 2023 · f6c5e85 · f6c5e85
1 parent fccc96e
commit f6c5e85
Show file tree

Hide file tree

Showing 3 changed files with 95 additions and 57 deletions.
diff --git a/src/liquid/components/ld-sidenav/ld-sidenav.tsx b/src/liquid/components/ld-sidenav/ld-sidenav.tsx
@@ -452,9 +452,12 @@ export class LdSidenav {
   private activeSubnavContainsIconsOrHasBack = () => {
     // Return true if we have a back button or a nav items as a direct child.
     if (
-      Array.from(this.el.children).filter((child) =>
-        ['LD-SIDENAV-BACK', 'LD-SIDENAV-NAVITEM'].includes(child.tagName)
-      )
+      Array.from(this.el.children).filter(
+        (child) =>
+          child.tagName === 'LD-SIDENAV-BACK' ||
+          (child.tagName === 'LD-SIDENAV-NAVITEM' &&
+            !(child as HTMLLdSidenavNavitemElement).mode)
+      ).length
     ) {
       return true
     }