@kolkrabbi/kol-component 0.47.1 → 0.49.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kolkrabbi/kol-component",
-  "version": "0.47.1",
+  "version": "0.49.0",
   "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/organisms/ContentCollection.jsx

@@ -13,15 +13,33 @@ import usePrefersReducedMotion from '../hooks/usePrefersReducedMotion.js'
  * ponytail: form switch re-mounts with a stagger, no FLIP — add FLIP
  * (measure → invert → play) when a consumer needs card↔row to tween.
  *
+ * BOTH FORMS ARE GRIDS (user call 2026-08-15: *"layout always wraps the items
+ * in a grid, at least 99% of the time"*). `list` is not a single column — it is
+ * the same wall with WIDER tracks, so a compact row gets the room its fields
+ * need. The shipped consumers already did this (`repeat(4, 1fr)` for list,
+ * `repeat(6, 1fr)` for grid); what they got wrong was FORCING the count.
+ *
+ * The count derives from the wall's OWN width, never the viewport, and never a
+ * fixed track count — 05-layout-systems.md, the card-wall law: *"the shell
+ * rails eat width that viewport breakpoints can't see, so a forced count
+ * compresses every card"* (user call 2026-08-09). Hence `auto-fill` +
+ * `minmax()`, with the law's 20rem minimum as the default.
+ *
  * @param {string}  form     'grid' | 'list'
- * @param {string}  min      grid track minimum (CSS length), grid form only
- * @param {number}  gap      px between items
+ * @param {string}  min      grid track minimum, card form (default 20rem — the law)
+ * @param {string}  listMin  grid track minimum, list form (default 30rem)
+ * @param {number}  gap      px between items — defaults PER FORM (grid 24,
+ *                           list 8), the values every shipped consumer
+ *                           re-typed as a ternary at its own call site. A
+ *                           single default meant the ternary survived the
+ *                           migration, which is half the duplication
  * @param {boolean} stagger  enter animation on/off (reduced motion wins)
  */
 export default function ContentCollection({
   form = 'grid',
-  min = '12rem',
-  gap = 16,
+  min = '20rem',
+  listMin = '30rem',
+  gap,
   stagger = true,
   children,
   className = '',
@@ -29,16 +47,19 @@ export default function ContentCollection({
   const reduced = usePrefersReducedMotion()
   const animate = stagger && !reduced
   const items = Array.isArray(children) ? children.flat() : [children]
+  /* ruled 2026-08-15 — the shipped 24/8 become the defaults, so `gap` is only
+   * passed when a consumer genuinely differs from the house. */
+  const px = gap ?? (form === 'list' ? 8 : 24)
 
   return (
     <ul
       key={form}
       className={`m-0 list-none p-0 ${className}`.trim()}
-      style={
-        form === 'grid'
-          ? { display: 'grid', gridTemplateColumns: `repeat(auto-fill, minmax(${min}, 1fr))`, gap: `${gap}px` }
-          : { display: 'flex', flexDirection: 'column', gap: `${gap}px` }
-      }
+      style={{
+        display: 'grid',
+        gridTemplateColumns: `repeat(auto-fill, minmax(${form === 'list' ? listMin : min}, 1fr))`,
+        gap: `${px}px`,
+      }}
     >
       {items.map((child, i) =>
         child == null ? null : (