jamdesk 1.1.191 → 1.1.192

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jamdesk",
-  "version": "1.1.191",
+  "version": "1.1.192",
   "description": "CLI for Jamdesk — build, preview, and deploy documentation sites from MDX. Dev server with hot reload, 50+ components, OpenAPI support, AI search, and Mintlify migration",
   "keywords": [
     "jamdesk",

package/vendored/components/navigation/Sidebar.tsx

@@ -815,9 +815,15 @@ export function Sidebar({ config, layout = 'header-logo', tabsPosition: tabsPosi
 
   const showTopDivider = resolvedNav.externalAnchors.length > 0;
   const hasTopTabsBar = getTabsFromConfig(config, pathname || undefined).length > 1 && !showTabsInSidebar;
+  // `lg:max-h-full` caps the sidebar at the height of the layout row that holds
+  // it — LayoutWrapper has already subtracted the banner, the header and the
+  // tabs bar from that row. Without the cap, `self-start` sizes the <aside> to
+  // its content and the theme's scroller has to guess those heights off 100vh;
+  // the guess (57px header) was 7px short of the 64px the header actually
+  // renders, so the last nav item sat under the row's `lg:overflow-hidden`.
   const desktopSidebarClasses = layout === 'sidebar-logo'
     ? 'lg:fixed lg:top-[var(--jd-banner-height,0px)] lg:left-0 lg:bottom-0 lg:z-40'
-    : 'lg:sticky lg:self-start lg:z-30 lg:flex-shrink-0';
+    : 'lg:sticky lg:self-start lg:max-h-full lg:z-30 lg:flex-shrink-0';
 
   return (
     <>

package/vendored/themes/halo/variables.css

@@ -575,19 +575,22 @@ body[data-theme="halo"] {
    height, and its <aside> is `lg:self-start` (height = content), so without a
    cap the nav grows unbounded and never scrolls once it's taller than the
    viewport. jam and nebula both carry this exact rule; halo's full-bleed rewrite
-   dropped it. Header is 57px tall; a top tab strip adds another 57px (114px). */
+   dropped it. */
 @media (min-width: 1024px) {
   body[data-theme="halo"] .sidebar-scroll {
     position: sticky !important;
-    flex: 0 0 auto !important;
-  }
-  body[data-theme="halo"] .sidebar-scroll[data-has-tabs="true"] {
-    top: 114px !important;
-    max-height: calc(100vh - 114px - var(--jd-banner-height, 0px)) !important;
-  }
-  body[data-theme="halo"] .sidebar-scroll[data-has-tabs="false"] {
-    top: 57px !important;
-    max-height: calc(100vh - 57px - var(--jd-banner-height, 0px)) !important;
+    top: 0 !important;
+    /* Fill the layout row, which LayoutWrapper has already sized to the space
+       left over after the banner, the header and the tabs bar. The rule this
+       replaces measured those back off 100vh with the header hardcoded at 57px
+       — it renders 64px — so the scroller overran its row by exactly 7px on
+       every desktop viewport and the row's `lg:overflow-hidden` clipped the
+       last nav item. Pairs with `lg:max-h-full` on the <aside>, which caps the
+       (content-sized, `self-start`) sidebar at the row height without changing
+       how a short nav renders. */
+    flex: 1 1 auto !important;
+    min-height: 0 !important;
+    max-height: none !important;
   }
 }
 

package/vendored/themes/jam/variables.css

@@ -721,21 +721,24 @@ body[data-theme="jam"] .sidebar-scroll .sidebar-nav-groups > div:first-child {
   }
 }
 
-/* Sidebar scroll container positioning for header-logo layout */
-/* The sidebar-scroll div needs constrained height to scroll independently */
-/* When tabs exist at top, sidebar needs more top offset (header 57px + tabs 57px = 114px) */
+/* Sidebar scroll container height cap for the header-logo layout — REQUIRED
+   for a tall nav to scroll. The shared markup gives .sidebar-scroll
+   `overflow-y-auto` but no bounded height, so the cap has to come from here. */
 @media (min-width: 1024px) {
   body[data-theme="jam"] .sidebar-scroll {
     position: sticky !important;
-    flex: 0 0 auto !important;
-  }
-  body[data-theme="jam"] .sidebar-scroll[data-has-tabs="true"] {
-    top: 114px !important;
-    max-height: calc(100vh - 114px - var(--jd-banner-height, 0px)) !important;
-  }
-  body[data-theme="jam"] .sidebar-scroll[data-has-tabs="false"] {
-    top: 57px !important;
-    max-height: calc(100vh - 57px - var(--jd-banner-height, 0px)) !important;
+    top: 0 !important;
+    /* Fill the layout row, which LayoutWrapper has already sized to the space
+       left over after the banner, the header and the tabs bar. The rule this
+       replaces measured those back off 100vh with the header hardcoded at 57px
+       — it renders 64px — so the scroller overran its row by exactly 7px on
+       every desktop viewport and the row's `lg:overflow-hidden` clipped the
+       last nav item. Pairs with `lg:max-h-full` on the <aside>, which caps the
+       (content-sized, `self-start`) sidebar at the row height without changing
+       how a short nav renders. */
+    flex: 1 1 auto !important;
+    min-height: 0 !important;
+    max-height: none !important;
   }
 }
 

package/vendored/themes/nebula/variables.css

@@ -214,21 +214,24 @@ body[data-theme="nebula"] .content-scroll::-webkit-scrollbar {
   display: none;
 }
 
-/* Sidebar scroll container positioning for header-logo layout */
-/* The sidebar-scroll div needs constrained height to scroll independently */
-/* When tabs exist at top, sidebar needs more top offset (header 57px + tabs 57px = 114px) */
+/* Sidebar scroll container height cap for the header-logo layout — REQUIRED
+   for a tall nav to scroll. The shared markup gives .sidebar-scroll
+   `overflow-y-auto` but no bounded height, so the cap has to come from here. */
 @media (min-width: 1024px) {
   body[data-theme="nebula"] .sidebar-scroll {
     position: sticky !important;
-    flex: 0 0 auto !important;
-  }
-  body[data-theme="nebula"] .sidebar-scroll[data-has-tabs="true"] {
-    top: 114px !important;
-    max-height: calc(100vh - 114px - var(--jd-banner-height, 0px)) !important;
-  }
-  body[data-theme="nebula"] .sidebar-scroll[data-has-tabs="false"] {
-    top: 57px !important;
-    max-height: calc(100vh - 57px - var(--jd-banner-height, 0px)) !important;
+    top: 0 !important;
+    /* Fill the layout row, which LayoutWrapper has already sized to the space
+       left over after the banner, the header and the tabs bar. The rule this
+       replaces measured those back off 100vh with the header hardcoded at 57px
+       — it renders 64px — so the scroller overran its row by exactly 7px on
+       every desktop viewport and the row's `lg:overflow-hidden` clipped the
+       last nav item. Pairs with `lg:max-h-full` on the <aside>, which caps the
+       (content-sized, `self-start`) sidebar at the row height without changing
+       how a short nav renders. */
+    flex: 1 1 auto !important;
+    min-height: 0 !important;
+    max-height: none !important;
   }
 }