Fixed an issue when linking to nested sidebars

website/src/theme/OWSidebar/sidebar-grid/sidebar-item/category-item.tsx

@@ -81,7 +81,7 @@ function CategoryItem(
         (isCurrentPage ||
           // OR auto-collapse is enabled
           (autoCollapseCategories &&
-            // And part the item is part of:
+            // And part of the item is part of:
             // - The active item's path
             // It is expanded
             (isActiveItem ||
@@ -137,8 +137,8 @@ function CategoryItem(
   const onInteraction = () => setExpand(!expanded);
 
   useEffect(() => {
-    if (isCurrentPage) setExpand(true);
-  }, [isCurrentPage, setExpand]);
+    if (isActiveItem) setExpand(true);
+  }, [isActiveItem, setExpand]);
 
   // ---------------------------------------------------------------------------
 

website/src/theme/OWSidebar/sidebar-grid/sidebar-item/utils/sidebar-link-hook.tsx

@@ -14,23 +14,27 @@ export default function SidebarLinkHook(
   initialHref: string,
   path: string,
 ): {
-  isMobile: boolean;
-  collapsed: boolean;
-  // Is this a parent of the current page
-  isActiveItem: boolean;
-  // Is this the current page
-  isCurrentPage: boolean;
-  // Is this a link that leads outside the local website?
+  /** Is this running within a mobile sidebar context */
+  isMobile: boolean
+  /** Is this currently contained within a collapsed context */
+  collapsed: boolean,
+  /** Is this a parent of the current page */
+  isActiveItem: boolean,
+  /** Is this the current page */
+  isCurrentPage: boolean,
+  /** Is this a link that leads outside the local website? */
   isExternalLink: boolean;
 } {
   return {
+    /** Is this running within a mobile sidebar context */
     isMobile: useContext(MobileContext),
+    /** Is this currently contained within a collapsed context */
     collapsed: useContext(CollapseContext),
-    // Is this a parent of the current page
+    /** Is this a parent of the current page */
     isActiveItem: isActiveSidebarItem(item, path),
-    // Is this the current page
+    /** Is this the current page */
     isCurrentPage: isSamePath(initialHref, path),
-    // Is this a link that leads outside the local website?
+    /** Is this a link that leads outside the local website? */
     isExternalLink: !(initialHref ? isInternalUrl(initialHref) : true),
   };
 }