@kolkrabbi/kol-workshop 0.2.1 → 0.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kolkrabbi/kol-workshop",
-  "version": "0.2.1",
+  "version": "0.3.1",
   "private": false,
   "description": "KOL workshop/docs system — a handrolled markdown engine, search, tag system (incl. a d3 tag graph), docs viewer, and docs shell. Sits above @kolkrabbi/kol-{theme,icons,component,framework}; content is consumer-injected (the package never globs docs).",
   "license": "MIT",
@@ -15,9 +15,9 @@
     "d3": "^7.9.0",
     "@kolkrabbi/kol-brand": "^0.1.2",
     "@kolkrabbi/kol-icons": "0.8.10",
-    "@kolkrabbi/kol-framework": "0.6.2",
-    "@kolkrabbi/kol-theme": "0.12.0",
-    "@kolkrabbi/kol-component": "0.12.19"
+    "@kolkrabbi/kol-component": "0.13.1",
+    "@kolkrabbi/kol-framework": "0.6.4",
+    "@kolkrabbi/kol-theme": "0.12.2"
   },
   "peerDependencies": {
     "react": "^18.3.0 || ^19.0.0",

package/src/shell/ShellLayout.jsx

@@ -46,7 +46,7 @@ const TocColumn = ({ children }) => (
   </aside>
 )
 
-const ShellLayout = ({ routes = [], basePath = '/', brandLogoSrc, brandLogoAlt = '', renderSidebar, searchItems, defaultTocContent, isActive: isActiveProp }) => {
+const ShellLayout = ({ routes = [], basePath = '/', brand: brandProp, brandLogoSrc, brandLogoAlt = '', renderSidebar, searchItems, defaultTocContent, isActive: isActiveProp, actions }) => {
   const [isNavDrawerOpen, setIsNavDrawerOpen] = useState(false)
   const [navCollapsed, setNavCollapsed] = useState(false)
   const [tocCollapsed, setTocCollapsed] = useState(false)
@@ -104,7 +104,10 @@ const ShellLayout = ({ routes = [], basePath = '/', brandLogoSrc, brandLogoAlt =
       navigate(item.href)
     }
   }
-  const brand = brandLogoSrc ? (
+  /* `brand` takes any node — a consumer that isn't the workshop needs its own
+   * wordmark, and brandLogoSrc only ever accepted an <img>. Falls back to the
+   * logo-src form, then to the KOLKRABBI + WORKSHOP pair. */
+  const brand = brandProp ?? (brandLogoSrc ? (
     <Link to={basePath} className="shell-header-logo flex items-center text-emphasis">
       <img src={brandLogoSrc} alt={brandLogoAlt} className="h-6 w-auto" />
     </Link>
@@ -120,11 +123,16 @@ const ShellLayout = ({ routes = [], basePath = '/', brandLogoSrc, brandLogoAlt =
         <Asset name="wordmark-workshop" title="Workshop" className="inline-flex [&>svg]:h-6 [&>svg]:w-auto" />
       </Link>
     </>
-  )
+  ))
+  /* Consumer `actions` sit BEFORE the search trigger in row 1 — a showcase
+   * needs its own controls (repo link, etc.) beside the shell's. */
   const searchTrigger = (
-    <Tooltip label="Search">
-      <Button variant="ghost" quiet iconOnly="search" iconSize={18} onClick={() => setIsSearchOpen(true)} aria-label="Search" />
-    </Tooltip>
+    <>
+      {actions}
+      <Tooltip label="Search">
+        <Button variant="ghost" quiet iconOnly="search" iconSize={18} onClick={() => setIsSearchOpen(true)} aria-label="Search" />
+      </Tooltip>
+    </>
   )
 
   /* searchItems ({ id, label, href, sectionLabel?, tags?, headings?, keywords? })
@@ -172,7 +180,11 @@ const ShellLayout = ({ routes = [], basePath = '/', brandLogoSrc, brandLogoAlt =
             * overscroll-none stops chaining between regions and the edge bounce. */}
           <div className="flex-1 overflow-hidden">
             <div className="h-full w-full px-4 md:px-5 lg:px-6">
-              <div className={`shell-content-grid grid gap-8 ${gridCols} h-full min-h-0`} data-layout={layoutType}>
+              {/* gap lives in .shell-content-grid (theme) — a gap-8 utility here
+                * outranks the layered theme rule at every width and killed the
+                * 48px wide step (ARCHITECTURE §5: component geometry in its own
+                * rule, never a utility racing it). */}
+              <div className={`shell-content-grid grid ${gridCols} h-full min-h-0`} data-layout={layoutType}>
                   {showNav && (
                     <NavColumn>
                       {renderSidebar ? renderSidebar({}) : <ShellSidebar routes={routes} basePath={basePath} />}