diff --git a/docs/audit/theming/README.md b/docs/audit/theming/README.md index 9d356c8ef..362eec7d3 100644 --- a/docs/audit/theming/README.md +++ b/docs/audit/theming/README.md @@ -10,9 +10,9 @@ - **Medido**: 2026-08-21 · **162 recetas** con CSS + **8 componentes sin receta** = 170 fichas, el árbol entero de `eidos/components/` - **La pregunta**: ¿cuánto de la apariencia de cada componente puede cambiar un tema **sin tocar el sistema ni la receta**? -- **Alcance global**: **48%** — 2257 de 4676 knobs pasan por un token público del componente -- **Reparto**: público 2257 · privado 638 · global 1204 · literal 577 · sistema transversal 509 _(fuera del ratio)_ -- **Sin token público propio**: 41 · **alcance < 20 %**: 35 · **alcance 100 %**: 8 · **con eje `size`**: 62 +- **Alcance global**: **48%** — 2265 de 4676 knobs pasan por un token público del componente +- **Reparto**: público 2265 · privado 636 · global 1198 · literal 577 · sistema transversal 509 _(fuera del ratio)_ +- **Sin token público propio**: 41 · **alcance < 20 %**: 34 · **alcance 100 %**: 8 · **con eje `size`**: 62 ## Cómo se lee @@ -108,7 +108,6 @@ La columna «contrato» cuenta las claves **públicas** del bloque del component | [month-grid](./month-grid.md) | 0% | 61 | 0 | 7 | 0 | 5 | 49 | 0 | – | | [year-grid](./year-grid.md) | 0% | 61 | 0 | 7 | 0 | 5 | 49 | 0 | – | | [field-langs](./field-langs.md) | 0% | 41 | 0 | 0 | 31 | 8 | 2 | 0 | – | -| [drag-drop](./drag-drop.md) | 0% | 16 | 0 | 3 | 6 | 0 | 7 | 1 | – | | [kbd](./kbd.md) | 0% | 14 | 0 | 1 | 9 | 4 | 0 | 0 | – | | [link](./link.md) | 0% | 14 | 0 | 4 | 2 | 5 | 3 | 0 | – | | [code](./code.md) | 0% | 13 | 0 | 0 | 3 | 4 | 6 | 0 | – | @@ -234,6 +233,7 @@ La columna «contrato» cuenta las claves **públicas** del bloque del component | [media-player](./media-player.md) | 88% | 49 | 43 | 0 | 0 | 6 | 0 | 61 | – | | [tabs](./tabs.md) | 88% | 76 | 66 | 6 | 2 | 1 | 1 | 79 | y | | [gradient-picker](./gradient-picker.md) | 88% | 27 | 23 | 0 | 2 | 1 | 1 | 25 | – | +| [drag-drop](./drag-drop.md) | 89% | 16 | 8 | 1 | 0 | 0 | 7 | 10 | – | | [chat-typing](./chat-typing.md) | 89% | 9 | 8 | 0 | 1 | 0 | 0 | 7 | – | | [breadcrumb](./breadcrumb.md) | 90% | 23 | 18 | 2 | 0 | 0 | 3 | 28 | y | | [card-group](./card-group.md) | 90% | 22 | 18 | 0 | 0 | 2 | 2 | 20 | – | diff --git a/docs/audit/theming/drag-drop.md b/docs/audit/theming/drag-drop.md index 4cf11d27b..59d8f4d89 100644 --- a/docs/audit/theming/drag-drop.md +++ b/docs/audit/theming/drag-drop.md @@ -5,31 +5,22 @@ > Vista de conjunto: [README](./README.md) · método y protocolo: > [`PLAN-theming.md`](../../process/PLAN-theming.md) §1, §2, §7. -- **Medido**: 2026-08-20 · **Alcance**: **0%** — 0 de 9 knobs por token público -- **Knobs de apariencia**: 16 — público 0 · privado 3 · global 6 · literal 0 · sistema 7 _(fuera del ratio)_ -- **Contrato hoy** (`lib/recipes/base.ts`): 1 pública(s) — `preview-z` · 2 privada(s) forward — `_palette-solid`, `_palette-element` +- **Medido**: 2026-08-21 · **Alcance**: **89%** — 8 de 9 knobs por token público +- **Knobs de apariencia**: 16 — público 8 · privado 1 · global 0 · literal 0 · sistema 7 _(fuera del ratio)_ +- **Contrato hoy** (`lib/recipes/base.ts`): 10 pública(s) — `preview-padding-block`, `preview-padding-inline`, `preview-radius`, `preview-border-width`, `preview-bg`, `preview-fg`, `preview-font-family`, `preview-font-size`, `preview-shadow`, `preview-z` · 2 privada(s) forward — `_palette-solid`, `_palette-element` - **Eje `size`**: no · **ficheros**: `drag-drop.css` ## 1. Knobs fuera de alcance -### 1.1 Directo a primitivo global (6) +### 1.1 Directo a primitivo global (0) -| # | fichero:línea | selector | propiedad | valor | -| ---: | --- | --- | --- | --- | -| 1 | `drag-drop.css:87` | `[data-drag-drop-preview]` | `padding` | `var(--space-2) var(--space-3)` | -| 2 | `drag-drop.css:88` | `[data-drag-drop-preview]` | `border-radius` | `var(--radius-md)` | -| 3 | `drag-drop.css:90` | `[data-drag-drop-preview]` | `background` | `var(--color-surface-default)` | -| 4 | `drag-drop.css:94` | `[data-drag-drop-preview]` | `color` | `var(--color-content-primary)` | -| 5 | `drag-drop.css:95` | `[data-drag-drop-preview]` | `font-family` | `var(--style-label-font-family)` | -| 6 | `drag-drop.css:96` | `[data-drag-drop-preview]` | `font-size` | `var(--font-size-sm)` | +_Ninguno._ -### 1.2 A través de un privado (3) +### 1.2 A través de un privado (1) | # | fichero:línea | selector | propiedad | valor | | ---: | --- | --- | --- | --- | | 1 | `drag-drop.css:68` | `[data-drag-drop-droppable][data-dragover]` | `background` | `var(--_drag-drop-palette-element)` | -| 2 | `drag-drop.css:89` | `[data-drag-drop-preview]` | `border` | `var(--border-width) solid var(--_drag-drop-palette-solid)` | -| 3 | `drag-drop.css:91` | `[data-drag-drop-preview]` | `box-shadow` | `var(--shadow-raised), 0 0 0 1px var(--_drag-drop-palette-element)` | ### 1.3 Literales (0) @@ -57,7 +48,7 @@ Consumidos y **no declarados en el CSS** (vienen de `base.ts` o de un estilo inl ## 4. Propuesta de corrección -### 4.1 Tokens a declarar en `lib/recipes/base.ts` (5) +### 4.1 Tokens a declarar en `lib/recipes/base.ts` (0) Valor **verbatim** del CSS de hoy: el default no se mueve, sólo cambia quién puede moverlo. Nombres derivados de recipe-contract §1 (ejes lógicos, talla @@ -67,17 +58,10 @@ no distingue lo que debería — se marca `⚠`. | token (`--drag-drop-…`) | scope TSC | valor propuesto | usos | | --- | --- | --- | ---: | -| `preview-radius` | `root` | `var(--radius-md)` | 1 | -| `preview-bg` | `root` | `var(--color-surface-default)` | 1 | -| `preview-fg` | `root` | `var(--color-content-primary)` | 1 | -| `preview-font-family` | `root` | `var(--style-label-font-family)` | 1 | -| `preview-font-size` | `root` | `var(--font-size-sm)` | 1 | -### 4.2 Sin nombre mecánico (4) +### 4.2 Sin nombre mecánico (1) -- **⚠ decisión: el privado que alimenta este knob no se declara en el CSS (viene de `base.ts` o de un estilo inline) — hay que resolverlo antes de nombrarlo** — 2: `background`, `box-shadow`. -- **⚠ decisión: `padding` es shorthand o eje físico — hay que partirlo en ejes lógicos antes de nombrarlo (recipe-contract §1, R-4.4)** — 1: `padding`. -- **⚠ decisión: `border` es shorthand o eje físico — hay que partirlo en ejes lógicos antes de nombrarlo (recipe-contract §1, R-4.4)** — 1: `border`. +- **⚠ decisión: el privado que alimenta este knob no se declara en el CSS (viene de `base.ts` o de un estilo inline) — hay que resolverlo antes de nombrarlo** — 1: `background`. ### 4.4 Lo que hay que comprobar a mano (PLAN-theming §1.3 · §7.4) @@ -91,6 +75,27 @@ no distingue lo que debería — se marca `⚠`. -_(pendiente — lo escribe el autor; se conserva al regenerar)_ +**Medido 2026-08-21** (sonda ×2 = 0 diffs en 1.827 valores · 7 estados). Es un +componente pequeño con una anatomía clara: **de sus 16 knobs, SIETE ya son +sistema** (opacidades de fantasma y deshabilitado, anillos de foco, el anillo +`inset` de los droppables) y quedan nueve puntuables, seis de ellos en el +PREVIEW. + +1. **Casi todo lo tokenizable vive en `[data-drag-drop-preview]`**, el fantasma + que sigue al puntero. Existe SÓLO durante el arrastre, así que ni la sonda + ni el guard lo ven en reposo: se verifica forzando el nodo. +2. **Los dos forwards de paleta se quedan privados** (THM-2). El fondo del + `[data-dragover]` los lee a pelo y no admite un público sin duplicar la + regla; el `border` y el `box-shadow` del preview los combinan con una + anchura y una sombra que SÍ son knobs, así que esas dos declaraciones pasan + a alcance con `preview-border-width` y `preview-shadow`. +3. **El `--ring-inset-*` de los droppables no se toca**: es el vocabulario del + anillo del sistema, que la receta ALIMENTA con su acento + (`--ring-inset-color: var(--_drag-drop-palette-element)`). Acuñar un + `droppable-ring-*` sería vocabulario paralelo sobre un sistema transversal. +4. **`preview-z` ya existía** en el contrato — es el único público que este + componente tenía, y se conserva tal cual. +5. El `padding` del preview es un shorthand de dos valores → dos knobs + lógicos (`preview-padding-block` + `preview-padding-inline`). diff --git a/scripts/theming-sentinel-exceptions.ts b/scripts/theming-sentinel-exceptions.ts index 4a615d840..82bd902cb 100644 --- a/scripts/theming-sentinel-exceptions.ts +++ b/scripts/theming-sentinel-exceptions.ts @@ -114,6 +114,28 @@ export const SENTINEL_EXCEPTIONS: Record> = { 'sentinel-spinner-reduced-duration': 'behind prefers-reduced-motion; emulated reduce context → reaches (4s → 11.5s)' }, + 'drag-drop': { + 'preview-padding-block': + 'the drag PREVIEW is created only while a drag is in flight (0 nodes at rest, counted); forced → reaches (8 → 1234px)', + 'preview-padding-inline': + 'the drag PREVIEW is created only while a drag is in flight (0 nodes at rest, counted); forced → reaches (12 → 1235px)', + 'preview-radius': + 'the drag PREVIEW is created only while a drag is in flight (0 nodes at rest, counted); forced → reaches (6 → 1236px)', + 'preview-border-width': + 'the drag PREVIEW is created only while a drag is in flight (0 nodes at rest, counted); forced → reaches (1 → 9px)', + 'preview-bg': + 'the drag PREVIEW is created only while a drag is in flight (0 nodes at rest, counted); forced → reaches (rgb(1,2,3))', + 'preview-fg': + 'the drag PREVIEW is created only while a drag is in flight (0 nodes at rest, counted); forced → reaches (rgb(4,5,6))', + 'preview-font-family': + 'the drag PREVIEW is created only while a drag is in flight (0 nodes at rest, counted); forced → reaches (Zapfino)', + 'preview-font-size': + 'the drag PREVIEW is created only while a drag is in flight (0 nodes at rest, counted); forced → reaches (14 → 99px)', + 'preview-shadow': + 'the drag PREVIEW is created only while a drag is in flight (0 nodes at rest, counted); forced → reaches (0 0 0 7px rgb(7,8,9))', + 'preview-z': + 'the drag PREVIEW is created only while a drag is in flight (0 nodes at rest, counted); forced → reaches (100 → 4321)' + }, 'link-preview': { 'trigger-fg': 'the panel opens on HOVER, so measuring it REQUIRES the pointer on the trigger — and hover is exactly the state whose rule overrides the resting ink. Structural, not a demo gap: the hover token (`hover-trigger-fg`) reads live in the same run', diff --git a/src/uix/eidos/components/drag-drop/README.md b/src/uix/eidos/components/drag-drop/README.md index 69f28a948..e2edfaccc 100644 --- a/src/uix/eidos/components/drag-drop/README.md +++ b/src/uix/eidos/components/drag-drop/README.md @@ -68,6 +68,32 @@ Referencias: [dnd-kit](https://dndkit.com/) · - **`Preview` como parte**: el fantasma es anatomía themeable (no un clon screenshot del navegador). +## Talla y tema + +**Guard R-5.4** (`npm run theming:sentinel -- drag-drop `): 0/10 en vivo, +y los diez adjudicados con su medición en +`scripts/theming-sentinel-exceptions.ts` — todos pertenecen al PREVIEW, que +sólo existe mientras hay un arrastre en curso (0 nodos en reposo, contados). +Forzando el nodo, los diez mueven su propiedad. Corrido 2026-08-21. + +Contrato en `lib/recipes/base.ts > drag-drop` (10 claves; `preview-z` ya +estaba). Alcance del censo: **89 %** (era 0 %). + +**Siete de sus dieciséis knobs YA eran sistema** y no entran al ratio: las +opacidades de fantasma y deshabilitado, los dos anillos de foco y el anillo +`inset` de los droppables. Quedan nueve puntuables, seis de ellos en el +preview. + +**Los dos forwards de paleta siguen privados** (THM-2). El fondo del +`[data-dragover]` los lee a pelo y no admite un público sin duplicar la regla — +es el 11 % que falta. El `border` y el `box-shadow` del preview sí los combinan +con una anchura y una sombra que son knobs propios, así que esas dos +declaraciones entran al alcance. + +**El `--ring-inset-*` de los droppables no se toca**: es vocabulario del anillo +del SISTEMA, que la receta se limita a alimentar con su acento. Acuñar un +`droppable-ring-*` sería vocabulario paralelo sobre un transversal. + ## Gaps | Gap | Disposición | Detalle | diff --git a/src/uix/eidos/components/drag-drop/drag-drop.css b/src/uix/eidos/components/drag-drop/drag-drop.css index 5df1fb6f3..39b92ca6b 100644 --- a/src/uix/eidos/components/drag-drop/drag-drop.css +++ b/src/uix/eidos/components/drag-drop/drag-drop.css @@ -84,15 +84,15 @@ [data-drag-drop-preview] { pointer-events: none; z-index: var(--drag-drop-preview-z); - padding: var(--space-2) var(--space-3); - border-radius: var(--radius-md); - border: var(--border-width) solid var(--_drag-drop-palette-solid); - background: var(--color-surface-default); + padding: var(--drag-drop-preview-padding-block) var(--drag-drop-preview-padding-inline); + border-radius: var(--drag-drop-preview-radius); + border: var(--drag-drop-preview-border-width) solid var(--_drag-drop-palette-solid); + background: var(--drag-drop-preview-bg); box-shadow: - var(--shadow-raised), + var(--drag-drop-preview-shadow), 0 0 0 1px var(--_drag-drop-palette-element); - color: var(--color-content-primary); - font-family: var(--style-label-font-family); - font-size: var(--font-size-sm); + color: var(--drag-drop-preview-fg); + font-family: var(--drag-drop-preview-font-family); + font-size: var(--drag-drop-preview-font-size); white-space: nowrap; } diff --git a/src/uix/eidos/generated/base.css b/src/uix/eidos/generated/base.css index fbd5c67d9..5b22b63f5 100644 --- a/src/uix/eidos/generated/base.css +++ b/src/uix/eidos/generated/base.css @@ -4266,6 +4266,15 @@ --command-separator-height: var(--border-width); --command-separator-margin-block: var(--space-1); --command-separator-bg: var(--color-neutral-separator); + --drag-drop-preview-padding-block: var(--space-2); + --drag-drop-preview-padding-inline: var(--space-3); + --drag-drop-preview-radius: var(--radius-md); + --drag-drop-preview-border-width: var(--border-width); + --drag-drop-preview-bg: var(--color-surface-default); + --drag-drop-preview-fg: var(--color-content-primary); + --drag-drop-preview-font-family: var(--style-label-font-family); + --drag-drop-preview-font-size: var(--size-sm-font-size); + --drag-drop-preview-shadow: var(--shadow-raised); --drag-drop-preview-z: var(--z-index-overlay-detached); --navigation-menu-content-z: var(--z-index-overlay-inline); --navigation-menu-indicator-w: 0px; diff --git a/src/uix/eidos/lib/recipes/base.ts b/src/uix/eidos/lib/recipes/base.ts index 6c4e12e7b..3071269b9 100644 --- a/src/uix/eidos/lib/recipes/base.ts +++ b/src/uix/eidos/lib/recipes/base.ts @@ -6989,6 +6989,19 @@ export const THEME_BASE_RECIPE_TOKENS = defineRecipes({ // DragDrop — the drag preview/ghost that tracks the cursor. Floats above // local content during an active drag. 'drag-drop': { + // The PREVIEW — the ghost that follows the pointer. It exists only during a + // drag, so neither the probe nor the guard sees it at rest. + 'preview-padding-block': 'var(--space-2)', + 'preview-padding-inline': 'var(--space-3)', + 'preview-radius': 'var(--radius-md)', + 'preview-border-width': 'var(--border-width)', + 'preview-bg': 'var(--color-surface-default)', + 'preview-fg': 'var(--color-content-primary)', + 'preview-font-family': 'var(--style-label-font-family)', + 'preview-font-size': 'var(--size-sm-font-size)', + // The lift under the ghost. The accent ring stacked on top of it comes from + // the palette forward and stays private. + 'preview-shadow': 'var(--shadow-raised)', 'preview-z': 'var(--z-index-overlay-detached)', // Selection palette (THM-2) — `data-color` sits on `[data-drag-drop-root]`. '_palette-solid': { diff --git a/web/routes/uix/components/drag-drop/+page.svelte b/web/routes/uix/components/drag-drop/+page.svelte index fecd8e0e5..e39a7b44c 100644 --- a/web/routes/uix/components/drag-drop/+page.svelte +++ b/web/routes/uix/components/drag-drop/+page.svelte @@ -8,10 +8,11 @@ import { compileMorfo } from '$uix/morfo'; import { dragDropMorfo } from '@/uix/morfo/components/drag-drop'; import { getActiveUix } from '$active-uix'; + import TokensPanel from '../../lib/TokensPanel.svelte'; const uix = getActiveUix(); - type Tab = 'live' | 'api' | 'morfo' | 'sema' | 'recipe' | 'a11y'; + type Tab = 'live' | 'api' | 'morfo' | 'sema' | 'recipe' | 'tokens' | 'a11y'; let tab = $state('live'); type Item = { id: string; label: string }; @@ -226,6 +227,9 @@ {events.length} + @@ -451,6 +455,10 @@ {/if} + {#if tab === 'tokens'} + + {/if} + {#if tab === 'a11y'}

Accessibility