@kolkrabbi/kol-component 0.32.5 → 0.33.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kolkrabbi/kol-component",
-  "version": "0.32.5",
+  "version": "0.33.1",
   "description": "KOL design-system components — atoms through organisms, emitting canonical kol-* classes. Pairs with @kolkrabbi/kol-theme for styling.",
   "license": "MIT",
   "type": "module",

package/src/molecules/FieldRow.jsx

@@ -1,10 +1,7 @@
 import { useState } from 'react'
 import { Icon } from '@kolkrabbi/kol-icons'
 import Input from '../atoms/Input'
-import Tag from '../atoms/Tag'
-import Button from '../atoms/Button'
 import { usePopover, PopoverPanel } from '../atoms/Popover'
-import { INDICATOR } from '../hooks/glyphLadders.js'
 import Dropdown from './Dropdown'
 
 /**
@@ -27,7 +24,7 @@ import Dropdown from './Dropdown'
 
 /* StatusChip — the interactive status control ("Live ▾" in the reference,
  * rebuilt to it 2026-08-09 after the user's screenshots: the chip carries a
- * TONE tint + a trailing caret at the INDICATOR rung, and the open panel is
+ * TONE tint + a trailing chevron, and the open panel is
  * the reference's grammar — current chip on top, hairline, then option rows
  * with a leading check column).
  * Chip taxonomy (Tag source, 2026-07-30): interactive → Tag, never Pill. The
@@ -57,19 +54,18 @@ export function StatusChip({ value, options = [], onChange, variant = 'neutral',
   const toneCls = activeOpt?.tone
     ? ` kol-status-chip--${activeOpt.tone}`
     : variant === 'primary' ? ' kol-status-chip--primary' : ''
-  /* BUILT ON THE COVER-FOCUS SPECS (user ruling 2026-08-09: "same component
-   * as cover focus"): the chip wears the Dropdown-sm trigger metrics — the
-   * kol-btn-sm box (py-1 px-3) and mono-12 type at its OWN 16px leading, so
-   * the pill stands the trigger's 24px (a leading-none had shrunk it to 20;
-   * user re-call same day: "like dropdown small in size"). The caret is a
-   * direct child of the flex row, centred by items-center; it cannot sit off
-   * the label's middle. */
+  /* CHIP BOX: px-2 is the user's exact value (2026-08-09: "x padding 2
+   * (8px)"). py-0.5 — the same-day py-0 collapsed the pill to the bare line
+   * box, and the reference chip ("why cant you make the chip look like
+   * this?") carries visible air; 2px is the minimum that restores it.
+   * mono-12 at its own 16px leading. The chevron is a direct child of the flex
+   * row, centred by items-center; it cannot sit off the label's middle. */
   const chip = (interactive) => (
     <button
       type="button"
       disabled={!interactive || undefined}
       aria-expanded={interactive ? open : undefined}
-      className={`kol-status-chip${toneCls} kol-mono-12 inline-flex items-center gap-1 rounded-full border-0 px-3 py-1${interactive ? ' cursor-pointer' : ''}`}
+      className={`kol-status-chip${toneCls} kol-mono-12 inline-flex items-center gap-2 rounded-full border-0 px-2 py-0.5${interactive ? ' cursor-pointer' : ''}`}
       onClick={interactive ? () => setOpen((o) => !o) : undefined}
     >
       {activeOpt?.label ?? value}
@@ -78,7 +74,7 @@ export function StatusChip({ value, options = [], onChange, variant = 'neutral',
           className={`inline-flex shrink-0 transition-transform duration-150${open ? ' rotate-180' : ''}`}
           aria-hidden="true"
         >
-          <Icon name="chevron-down" size={INDICATOR.sm} />
+          <Icon name="chevron-down" size={10} />
         </span>
       )}
     </button>
@@ -123,9 +119,10 @@ export function StatusChip({ value, options = [], onChange, variant = 'neutral',
   )
 }
 
-/* Media thumbnail — the reference panel's proportions (frame 39,
- * 2026-08-09): a generous rounded tile, remove × inset at its corner. */
-const THUMB_CLS = 'h-16 w-28 rounded-lg object-cover bg-fg-04'
+/* Media thumbnail — measured off the reference panel (2026-08-09): 136×72,
+ * remove × riding the corner. Radius is the system's 4px — the repo ships
+ * 4px or full, nothing between. */
+const THUMB_CLS = 'h-18 w-34 rounded object-cover bg-fg-04'
 
 export default function FieldRow({
   label,
@@ -149,7 +146,9 @@ export default function FieldRow({
       aria-label={label}
       disabled={disabled || !onPick}
       onClick={onPick}
-      className="flex h-16 w-28 items-center justify-center rounded-lg border border-fg-16 bg-fg-02 text-body transition-colors duration-150 enabled:cursor-pointer enabled:hover:text-emphasis disabled:opacity-60"
+      /* reference empty tile: FILLED, no border — a plain grey slab with the
+       * + centred (frame 2026-08-09) */
+      className="flex h-18 w-34 items-center justify-center rounded border-0 bg-fg-04 text-body transition-colors duration-150 enabled:cursor-pointer enabled:hover:text-emphasis disabled:opacity-60"
     >
       <Icon name="plus" size={16} />
     </button>
@@ -162,15 +161,16 @@ export default function FieldRow({
       onClick={onPick}
       className="kol-control kol-control--filled kol-control-sm kol-mono-12 w-full text-left text-meta enabled:cursor-pointer disabled:opacity-60"
     >
-      Choose file…
+      Choose File...
     </button>
   )
 
   let control = null
   if (type === 'text') {
     control = (
-      /* sm — record rows are compact chrome (the reference's density); md
-       * read as a form field ("input is large? why?", 2026-08-09). */
+      /* sm — the repo's control rung, no special box for this component
+       * (user ruling 2026-08-09: "why would this component be different
+       * than every other rule in the repo"). */
       <Input
         size="sm"
         value={value ?? ''}
@@ -183,40 +183,60 @@ export default function FieldRow({
   } else if (type === 'status') {
     control = <StatusChip value={value} options={options} onChange={onChange} />
   } else if (type === 'select') {
-    control = <Dropdown options={options} value={value} onChange={onChange} />
+    /* full-width — the reference's focus selects span the control column,
+     * caret at the far edge (panel frame 2026-08-09) */
+    control = <Dropdown options={options} value={value} onChange={onChange} className="w-full" />
   } else if (type === 'media') {
     control = value ? (
-      <span className="group relative inline-flex">
+      <span className="relative inline-flex">
         <img src={value.thumb ?? value.url} alt={value.name ?? ''} className={THUMB_CLS} />
         {onChange && (
-          <Button
-            variant="nav"
-            size="sm"
-            iconOnly="x"
-            iconSize={10}
+          /* the reference badge: ALWAYS visible, a small dark disc riding the
+           * tile's corner — scrim ink is theme-invariant over imagery (panel
+           * frame 2026-08-09; hover-reveal repealed) */
+          <button
+            type="button"
             aria-label={label}
-            className="absolute top-1 right-1 opacity-0 group-hover:opacity-100 focus-visible:opacity-100"
             onClick={() => onChange(null)}
-          />
+            className="absolute top-1 right-1 inline-flex h-5 w-5 cursor-pointer items-center justify-center rounded-full border-0 kol-overlay-scrim text-white"
+          >
+            <Icon name="x" size={10} />
+          </button>
         )}
       </span>
     ) : mediaEmpty
   } else if (type === 'file') {
     control = value ? (
-      /* kol-tag--data: a filename renders VERBATIM — the chip uppercase is
-       * for label strings, and transforming data misquotes it (2026-08-09). */
-      <Tag variant="secondary" size="sm" hash={false} className="kol-tag--data" onRemove={onChange ? () => onChange(null) : undefined}>
-        {value.name ?? value}
-      </Tag>
+      /* the reference's file value is a FULL-WIDTH filled row — filename at
+       * the leading edge (verbatim, data is never case-transformed), × at the
+       * far end (panel frame 2026-08-09; the kol-tag--data chip repealed) */
+      <span className="kol-control kol-control--filled kol-control-sm kol-mono-12 flex w-full items-center gap-2">
+        <span className="min-w-0 flex-1 truncate">{value.name ?? value}</span>
+        {onChange && (
+          <button
+            type="button"
+            aria-label={label}
+            onClick={() => onChange(null)}
+            className="inline-flex shrink-0 cursor-pointer border-0 bg-transparent p-0 text-body hover:text-emphasis"
+          >
+            <Icon name="x" size={12} />
+          </button>
+        )}
+      </span>
     ) : fileEmpty
   }
 
   return (
-    <div className="grid grid-cols-[minmax(0,10rem)_minmax(0,1fr)] items-center gap-x-6 py-4 border-b border-fg-08">
-      <div className="kol-mono-12 text-body">{label}</div>
+    /* reference rhythm (2026-08-09): roomy rows on hairlines, narrow label
+     * column, labels white; media rows top-align their label to the tile.
+     * Controls stay on the repo's sm rung — no special box here. */
+    /* .kol-field-row carries the grid anatomy in kol-theme (package chrome
+     * never rides arbitrary utilities — they can miss generation) */
+    <div className="kol-field-row items-center gap-x-6 py-3 border-b border-fg-08">
+      <div className={`kol-mono-12 text-emphasis${type === 'media' ? ' self-start pt-1' : ''}`}>{label}</div>
       <div className="min-w-0">{control}</div>
       {hint != null && (
-        <div className="col-start-2 kol-helper-10 text-meta pt-1 truncate">{hint}</div>
+        <div className="col-start-2 kol-helper-12 text-meta pt-2 truncate">{hint}</div>
       )}
     </div>
   )

package/src/molecules/ShellDrawer.jsx

@@ -1,5 +1,6 @@
 import { useEffect, useRef, useState } from 'react'
 import { createPortal } from 'react-dom'
+import { Icon } from '@kolkrabbi/kol-icons'
 import Button from '../atoms/Button.jsx'
 import usePrefersReducedMotion from '../hooks/usePrefersReducedMotion.js'
 
@@ -28,6 +29,7 @@ const FOCUSABLE =
  * @param {string}        side      'left' | 'right' — edge the panel slides from
  * @param {number|string} width     panel width (px number or CSS length); omit for full-width sheet
  * @param {ReactNode}     header    header-row content beside the close button (replaces the source's baked-in wordmark)
+ * @param {boolean}       backdrop  render the dimming scrim (default true); false = panel alone, no darken/blur, close via × / Esc
  * @param {ReactNode}     children  scrollable panel body
  * @param {string}        className extra classes on the panel
  */
@@ -38,6 +40,7 @@ export default function ShellDrawer({
   width,
   header,
   closeSide = 'end',
+  backdrop = true,
   children,
   className = '',
 }) {
@@ -134,26 +137,40 @@ export default function ShellDrawer({
 
   return createPortal(
     <>
-      <div
-        className={`fixed inset-0 z-[100] kol-overlay-scrim ${motionBackdrop}`}
-        onClick={onClose}
-        aria-hidden="true"
-      />
+      {backdrop && (
+        <div
+          className={`fixed inset-0 z-[100] kol-overlay-scrim ${motionBackdrop}`}
+          onClick={onClose}
+          aria-hidden="true"
+        />
+      )}
       <div
         ref={panelRef}
         role="dialog"
         aria-modal="true"
         tabIndex={-1}
-        className={`fixed inset-y-0 z-[200] flex max-w-full flex-col bg-surface-primary px-4 py-4 shadow-2xl outline-none md:px-5 lg:px-6 ${
+        className={`fixed inset-y-0 z-[200] flex max-w-full flex-col bg-surface-primary px-4 py-4 outline-none md:px-5 lg:px-6 ${backdrop ? 'shadow-2xl' : ''} ${
           side === 'right' ? 'right-0 border-l' : 'left-0 border-r'
         } border-fg-08 ${width == null ? 'w-full' : ''} ${motionPanel} ${className}`}
         style={width != null ? { width: typeof width === 'number' ? `${width}px` : width } : undefined}
       >
-        <div className="mb-6 flex items-center gap-4">
+        {/* closeSide="start": the reference sets the × glyph ~9px deeper than
+          * the label column's edge, with extra top air (both reference frames
+          * 2026-08-09 measure the same inset) */}
+        <div className={`mb-6 flex items-center gap-4${closeSide === 'start' ? ' pt-3 pl-2' : ''}`}>
           {/* closeSide="start" — the record-surface anatomy (reference frame,
             * 2026-08-09): × leads, the header's actions sit at the far end. */}
           {closeSide === 'start' && (
-            <Button variant="nav" size="md" iconOnly="x" iconSize={14} onClick={onClose} aria-label="Close" className="shrink-0" />
+            /* bare glyph, no container (user 2026-08-09: "the X close icon
+             * does not need a container") — body ink at rest, emphasis on hover */
+            <button
+              type="button"
+              aria-label="Close"
+              onClick={onClose}
+              className="inline-flex shrink-0 cursor-pointer border-0 bg-transparent p-0 text-body transition-colors duration-150 hover:text-emphasis"
+            >
+              <Icon name="x" size={18} />
+            </button>
           )}
           {header != null && <div className="min-w-0 flex-1">{header}</div>}
           {/* The box has an owner (2026-08-01). This hand-wrote the icon-button

package/src/organisms/RecordManager.jsx

@@ -4,6 +4,7 @@ import { Icon } from '@kolkrabbi/kol-icons'
 import Button from '../atoms/Button'
 import SearchInput from '../molecules/SearchInput'
 import ToggleCheckbox from '../atoms/ToggleCheckbox'
+import IconFrame from '../atoms/IconFrame'
 import ShellDrawer from '../molecules/ShellDrawer'
 import FieldRow, { StatusChip } from '../molecules/FieldRow'
 import { Tooltip } from '../atoms/Popover'
@@ -100,7 +101,14 @@ export default function RecordManager({
 
   const rowId = (row, i) => row.id ?? i
 
+  /* panel = 60% of the table area (user ruling 2026-08-09) — the surface's
+   * own width IS the table pane, measured at open. ponytail: not re-measured
+   * on live resize. */
+  const [panelW, setPanelW] = useState(null)
+
   const openRecord = (row) => {
+    const w = wrapRef.current?.getBoundingClientRect().width
+    setPanelW(w ? Math.round(w * 0.6) : null)
     setActive(row)
     onSelectRow?.(row)
   }
@@ -153,14 +161,22 @@ export default function RecordManager({
              * the text (user frames 2026-08-09). Hover-revealed per row. */
             <span className="flex items-center justify-between gap-2 min-w-0 max-w-full">
               <span className="truncate text-emphasis">{col.render ? col.render(row) : row[col.accessor]}</span>
-              <button
-                type="button"
+              {/* the ICON CONTAINER is IconFrame (2026-08-01 ruling), box-model
+                * proper (user 2026-08-09, "padding should control the size"):
+                * the box stays pinned, the glyph FILLS the content box, and
+                * padding is the only inset knob — p-[3px] on user instruction
+                * (2026-08-09, "use 3px … use [3px]"); arbitrary utilities have
+                * missed consumer generation before — if this renders unpadded
+                * in a consumer, this class is why. */}
+              <IconFrame
+                name="toggle-overlay"
+                variant="outline"
+                size="sm"
+                iconSize="100%"
                 aria-label="Toggle overlay"
-                className="shrink-0 inline-flex cursor-pointer rounded-md border border-fg-16 bg-transparent p-1 text-body opacity-0 transition-opacity duration-150 group-hover:opacity-100 focus-visible:opacity-100 hover:text-emphasis"
                 onClick={() => openRecord(row)}
-              >
-                <Icon name="toggle-overlay" size={18} />
-              </button>
+                className="p-[3px] opacity-0 group-hover:opacity-100 focus-visible:opacity-100"
+              />
             </span>
           ),
         }
@@ -181,12 +197,11 @@ export default function RecordManager({
         return {
           ...col,
           /* the reference's select cell ("Center ▾") is a PILL, not the square
-           * Dropdown trigger — the StatusChip apparatus wearing the Dropdown-
-           * primary fill (user frame 2026-08-09: "use status pill, but same
-           * look as dropdown primary"). Panel select fields stay Dropdown. */
+           * Dropdown trigger — the StatusChip apparatus in its NEUTRAL grey,
+           * the same chip Archived wears (user ruling 2026-08-09: "grey like
+           * 'archived'"). Panel select fields stay Dropdown. */
           render: (row) => (
             <StatusChip
-              variant="primary"
               value={row[col.accessor]}
               options={col.options ?? []}
               onChange={(next) => col.onSelectChange?.(row, next)}
@@ -200,8 +215,9 @@ export default function RecordManager({
           render: (row) => {
             const media = row[col.accessor]
             return media ? (
-              /* 2:1 landscape (user frame 2026-08-09: "more landscape") */
-              <img src={media.thumb ?? media.url} alt={media.name ?? ''} className="h-10 w-20 rounded object-cover bg-fg-04" />
+              /* measured off the reference frame: the tile is chip-height —
+               * ~24px tall, 2:1 — not a 40px block (user re-call 2026-08-09) */
+              <img src={media.thumb ?? media.url} alt={media.name ?? ''} className="h-6 w-12 rounded object-cover bg-fg-04" />
             ) : (
               <span className="kol-helper-10 text-meta">—</span>
             )
@@ -323,8 +339,9 @@ export default function RecordManager({
         open={!!active}
         onClose={() => setActive(null)}
         side="right"
-        width="min(37.5rem, 92vw)"
+        width={panelW ? `${panelW}px` : '50vw'}
         closeSide="start"
+        backdrop={false}
         header={
           /* reference header anatomy (frame 45, 2026-08-09): × leads the row
            * (closeSide above); every action clusters at the far end —
@@ -346,7 +363,9 @@ export default function RecordManager({
         }
       >
         {record && (
-          <div className="flex flex-col">
+          /* border-t: the reference rules EVERY seam — including above the
+           * first row; each FieldRow carries its own border-b (frame 2026-08-09) */
+          <div className="flex flex-col border-t border-fg-08">
             {fields.map((f) => (
               <FieldRow
                 key={f.key}