origam 2.18.10 → 2.18.12

package/dist/src/assets/css/tokens/dark.css

@@ -448,6 +448,8 @@
   --origam-chip__close---max-width: 18px;
   --origam-chip__close---user-select: none;
   --origam-chip__close---flex: 0 1 auto;
+  --origam-chip__prepend---margin-inline-end: 6px;
+  --origam-chip__append---margin-inline-start: 6px;
   --origam-chip__filter---transition-property: transform, opacity;
   --origam-chip__filter---transition-duration: var(--origam-motion__duration---normal);
   --origam-chip__filter---transition-timing-function: var(--origam-motion__easing---standard);
@@ -3190,6 +3192,8 @@
     --origam-chip__close---max-width: 18px;
     --origam-chip__close---user-select: none;
     --origam-chip__close---flex: 0 1 auto;
+    --origam-chip__prepend---margin-inline-end: 6px;
+    --origam-chip__append---margin-inline-start: 6px;
     --origam-chip__filter---transition-property: transform, opacity;
     --origam-chip__filter---transition-duration: var(--origam-motion__duration---normal);
     --origam-chip__filter---transition-timing-function: var(--origam-motion__easing---standard);

package/dist/src/assets/css/tokens/light.css

@@ -350,6 +350,8 @@
   --origam-chip__close---max-width: 18px;
   --origam-chip__close---user-select: none;
   --origam-chip__close---flex: 0 1 auto;
+  --origam-chip__prepend---margin-inline-end: 6px;
+  --origam-chip__append---margin-inline-start: 6px;
   --origam-chip__filter---transition-property: transform, opacity;
   --origam-chip__filter---transition-duration: var(--origam-motion__duration---normal);
   --origam-chip__filter---transition-timing-function: var(--origam-motion__easing---standard);

package/dist/src/assets/scss/tokens/_dark.scss

@@ -448,6 +448,8 @@
   --origam-chip__close---max-width: 18px;
   --origam-chip__close---user-select: none;
   --origam-chip__close---flex: 0 1 auto;
+  --origam-chip__prepend---margin-inline-end: 6px;
+  --origam-chip__append---margin-inline-start: 6px;
   --origam-chip__filter---transition-property: transform, opacity;
   --origam-chip__filter---transition-duration: var(--origam-motion__duration---normal);
   --origam-chip__filter---transition-timing-function: var(--origam-motion__easing---standard);
@@ -3190,6 +3192,8 @@
     --origam-chip__close---max-width: 18px;
     --origam-chip__close---user-select: none;
     --origam-chip__close---flex: 0 1 auto;
+    --origam-chip__prepend---margin-inline-end: 6px;
+    --origam-chip__append---margin-inline-start: 6px;
     --origam-chip__filter---transition-property: transform, opacity;
     --origam-chip__filter---transition-duration: var(--origam-motion__duration---normal);
     --origam-chip__filter---transition-timing-function: var(--origam-motion__easing---standard);

package/dist/src/assets/scss/tokens/_light.scss

@@ -350,6 +350,8 @@
   --origam-chip__close---max-width: 18px;
   --origam-chip__close---user-select: none;
   --origam-chip__close---flex: 0 1 auto;
+  --origam-chip__prepend---margin-inline-end: 6px;
+  --origam-chip__append---margin-inline-start: 6px;
   --origam-chip__filter---transition-property: transform, opacity;
   --origam-chip__filter---transition-duration: var(--origam-motion__duration---normal);
   --origam-chip__filter---transition-timing-function: var(--origam-motion__easing---standard);

package/dist/src/components/Btn/OrigamBtn.vue

@@ -440,6 +440,24 @@ defineExpose({
 	.origam-btn {
 		$this: &;
 
+		// ⛔ #933 — `display` was never declared here. The root then fell back
+		// to the UA default, which for the `<a>` / `<span>` tags `useLink`
+		// renders is `inline`. An inline box ignores `height`, `min-width` and
+		// vertical `padding`, and it stretches to the full line box — so a Btn
+		// dropped into a plain block parent (`<p>`, `<div>`) rendered as a
+		// full-width, 15px-tall strip. Every Btn in this DS *looked* correct
+		// only because its parent happened to be a flex/grid container, which
+		// blockifies the child: measured on the marketing site, all 9 visible
+		// Btn roots computed `display: block` with a `display: flex` parent,
+		// and the single Btn inside a `<p>` computed `inline` at 984px wide.
+		//
+		// `inline-block` is the value the rest of this rule already assumed —
+		// `vertical-align: middle` right below only has meaning on an
+		// inline-level box. It is also the safest: inside a flex/grid parent it
+		// blockifies to `block`, i.e. byte-for-byte today's computed value, so
+		// no existing layout moves. `&--block` still overrides it with `flex`.
+		display: inline-block;
+
 		position: var(--origam-btn---position, relative);
 		vertical-align: middle;
 		flex-shrink: 0;

package/dist/src/components/Chip/OrigamChip.vue

@@ -435,6 +435,44 @@ defineExpose({
 			display: inline-flex;
 		}
 
+		// #957 — `__close` owned both of its margin channels since forever;
+		// `__prepend` / `__append` owned NEITHER, so a chip with an icon
+		// rendered it glued to its label (measured: 0px, `margin-inline-end:
+		// 0px`, `gap: normal`, on all 8 identities) and no theme could fix it
+		// — the channel had never been opened. Default 6px is not a new design
+		// value: it is `--origam-chip__close---margin-inline-start`'s, i.e.
+		// the component's own precedent for "gutter between an affix and the
+		// label".
+		//
+		// ⛔ The gutter is CONDITIONAL on the label being non-empty, and that
+		// is the whole difficulty. `__content` has NO `v-if`: an icon-only
+		// chip (`<origam-chip prepend-icon="…"/>`, no text, no default slot)
+		// still renders `<div class="origam-chip__content">` holding the
+		// empty text node Vue emits for `{{ text }}`. An unconditional margin
+		// would therefore hang a 6px PHANTOM gutter off the trailing edge of
+		// every icon-only chip, in 217 places. Measured in Chromium, which is
+		// what makes `:empty` the right discriminator here:
+		//
+		//   content = "text"                  → :empty false → gutter 6px
+		//   content = <div></div>             → :empty true  → gutter 0px
+		//   content = empty text node (Vue!)  → :empty true  → gutter 0px
+		//   content = " " (one space)         → :empty false → gutter 6px
+		//
+		// `__prepend` needs `:has(+ …)` because the element it depends on is
+		// its FOLLOWING sibling; `__append` reads its PRECEDING sibling, so a
+		// plain `+` combinator suffices. Both keep the declaration on the
+		// element the token NAMES, so `--origam-chip__prepend---*` stays
+		// readable as "the prepend zone's own margin". Where `:has()` is
+		// unsupported the prepend gutter is simply absent — today's rendering,
+		// not a regression.
+		&__prepend:has(+ #{$this}__content:not(:empty)) {
+			margin-inline-end: var(--origam-chip__prepend---margin-inline-end, 6px);
+		}
+
+		&__content:not(:empty) + &__append {
+			margin-inline-start: var(--origam-chip__append---margin-inline-start, 6px);
+		}
+
 		&__close {
 			cursor: var(--origam-chip__close---cursor, pointer);
 			flex: var(--origam-chip__close---flex, 0 1 auto);

package/dist/src/components/List/OrigamList.vue

@@ -355,7 +355,16 @@ defineExpose({
 		}
 
 		&--nav {
-			--origam-list---padding-inline: 8px;
+			// ⛔ #934 — this used to declare `--origam-list---padding-inline`,
+			// a name NOTHING reads. A custom property is not a shorthand: it
+			// expands to nothing, so `padding-inline-start/end` above kept
+			// resolving to the sheet's `--origam-space---0`. Measured on the
+			// marketing appbar menu across all 8 identities: list inline
+			// padding `0px/0px`, so the rows ran edge-to-edge inside the menu
+			// and their hover surface was clipped by the menu's own corner
+			// radius. The two longhands are the names the base rule reads.
+			--origam-list---padding-inline-start: var(--origam-space---2, 8px);
+			--origam-list---padding-inline-end: var(--origam-space---2, 8px);
 			--origam-list-subheader---font-size: var(--origam-list__subheader---nav-font-size, 0.75rem);
 			--origam-list---indent-padding: var(--origam-list---indent-padding-nav, -8px);
 		}

package/dist/src/components/List/OrigamListItem.vue

@@ -626,7 +626,12 @@ defineExpose({
 		}
 
 		&--nav {
-			--origam-list-item---padding-inline: 8px;
+			// ⛔ #934 — same dead name as `.origam-list--nav` carried:
+			// `--origam-list-item---padding-inline` is read nowhere. The base
+			// rule reads the two longhands, inside a `calc()` that also folds
+			// in indent and density, so they go in here under the same names.
+			--origam-list-item---padding-inline-start: var(--origam-space---2, 8px);
+			--origam-list-item---padding-inline-end: var(--origam-space---2, 8px);
 
 			&:not(:only-child) {
 				--origam-list-item---margin-block-end: 4px;

package/dist/src/components/Switch/OrigamSwitch.vue

@@ -473,6 +473,44 @@ defineExpose({
           }
         }
       }
+
+      /*
+       * ⛔ MEASURED (Chromium, /why-origam demo, all 8 marketing identities,
+       * light + dark, checked + unchecked): `OrigamSelectionControl`'s shared
+       * `&--focus-visible` rule draws its outline on
+       * `.origam-selection-control__input` — a ~28px square hit-area that is
+       * a SIBLING of `.origam-switch-track` (rendered via the `default`
+       * slot), never a descendant. The track's `overflow: hidden` therefore
+       * can never clip that outline, and because the square hit-area is
+       * already taller than the track (24px default / 32px inset) and offset
+       * by the thumb's hardcoded ±10px `translateX`, the ring visibly
+       * detaches from the track: 8px past its top/bottom edge and 8px past
+       * whichever side the thumb isn't currently near (right when ON, left
+       * when OFF) on every identity, `material` included — its own
+       * screenshot simply wasn't captured while focused.
+       *
+       * For Checkbox/Radio the square hit-area IS the visible control, so
+       * the shared rule is correct there and stays untouched
+       * (`OrigamSelectionControl.vue` is not modified). For Switch the
+       * visible control is the elongated track, so the ring is moved onto
+       * `.origam-switch-track` instead — same two tokens the shared rule
+       * already reads (`--origam-border__width---2`,
+       * `--origam-color__border---focus`), just retargeted to the shape a
+       * switch actually is. Native `outline` already follows an element's
+       * own `border-radius` in every supported browser, so the ring traces
+       * the track's real shape (pill / rounded / square per `rounded` prop)
+       * with no extra CSS.
+       */
+      &--focus-visible {
+        :deep(.origam-selection-control__input) {
+          outline: none;
+        }
+
+        :deep(.origam-switch-track) {
+          outline: var(--origam-border__width---2, 2px) solid var(--origam-color__border---focus, currentColor);
+          outline-offset: var(--origam-space---1, 2px);
+        }
+      }
     }
 
     &.origam-input {