You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
svelte-kit-vice/docs/audit/theming/popover.md

131 lines
12 KiB

# popover — alcance de tema: análisis y propuesta
> Generado por `node --import tsx/esm scripts/theming-census.ts --report`.
> Lo **medido** y la **propuesta** se regeneran; el **Veredicto** (§5) se conserva.
> Vista de conjunto: [README](./README.md) · método y protocolo:
> [`PLAN-theming.md`](../../process/PLAN-theming.md) §1, §2, §7.
- **Medido**: 2026-08-22 · **Alcance**: **46%** — 22 de 48 knobs por token público
- **Knobs de apariencia**: 51 — público 22 · privado 4 · global 22 · literal 0 · sistema 3 _(fuera del ratio)_
- **Contrato hoy** (`lib/recipes/base.ts`): 36 pública(s) — `content-z`, `overlay-z`, `overlay-bg`, `trigger-font-size`, `gap`, `max-width`, `min-width-sm`, `min-width-md`, `min-width-lg`, `min-width-xl`, `padding-block-sm`, `padding-inline-sm`, `padding-block-md`, `padding-inline-md`, `padding-block-lg`, `padding-inline-lg`, `padding-block-xl`, `padding-inline-xl`, `font-size`, `font-family`, `font-weight`, `line-height`, `title-font-family`, `title-font-size`, `title-font-weight`, `title-line-height`, `title-fg`, `description-font-family`, `description-font-size`, `description-line-height`, `description-fg`, `radius`, `bg`, `fg`, `arrow-width`, `arrow-height`
- **Eje `size`**: sí · **ficheros**: `popover.css`
## 1. Knobs fuera de alcance
### 1.1 Directo a primitivo global (22)
| # | fichero:línea | selector | propiedad | valor |
| ---: | --- | --- | --- | --- |
| 1 | `popover.css:19` | `[data-popover-trigger]:not([data-archetype='field-trigger'])` | `height` | `var(--control-height-md)` |
| 2 | `popover.css:20` | `[data-popover-trigger]:not([data-archetype='field-trigger'])` | `padding-inline` | `var(--space-3)` |
| 3 | `popover.css:21` | `[data-popover-trigger]:not([data-archetype='field-trigger'])` | `border` | `var(--border-width) solid var(--color-border-default)` |
| 4 | `popover.css:22` | `[data-popover-trigger]:not([data-archetype='field-trigger'])` | `border-radius` | `var(--radius-md)` |
| 5 | `popover.css:23` | `[data-popover-trigger]:not([data-archetype='field-trigger'])` | `background` | `var(--color-surface-raised)` |
| 6 | `popover.css:24` | `[data-popover-trigger]:not([data-archetype='field-trigger'])` | `color` | `var(--color-content-primary)` |
| 7 | `popover.css:34` | `[data-popover-trigger]:not([data-archetype='field-trigger']):hover:not([disabled]):not([data-disabled])` | `background` | `var(--color-surface-overlay)` |
| 8 | `popover.css:35` | `[data-popover-trigger]:not([data-archetype='field-trigger']):hover:not([disabled]):not([data-disabled])` | `border-color` | `var(--color-border-strong)` |
| 9 | `popover.css:39` | `[data-popover-trigger]:not([data-archetype='field-trigger'])[data-state='open']` | `background` | `var(--color-surface-overlay)` |
| 10 | `popover.css:40` | `[data-popover-trigger]:not([data-archetype='field-trigger'])[data-state='open']` | `border-color` | `var(--color-border-strong)` |
| 11 | `popover.css:247` | `[data-popover-content][data-state='closed'][data-side='top'][data-last-action='saved']` | `border-bottom` | `var(--border-width-medium) solid var(--color-fulfill-element)` |
| 12 | `popover.css:248` | `[data-popover-content][data-state='closed'][data-side='bottom'][data-last-action='saved']` | `border-top` | `var(--border-width-medium) solid var(--color-fulfill-element)` |
| 13 | `popover.css:249` | `[data-popover-content][data-state='closed'][data-side='right'][data-last-action='saved']` | `border-left` | `var(--border-width-medium) solid var(--color-fulfill-element)` |
| 14 | `popover.css:250` | `[data-popover-content][data-state='closed'][data-side='left'][data-last-action='saved']` | `border-right` | `var(--border-width-medium) solid var(--color-fulfill-element)` |
| 15 | `popover.css:252` | `[data-popover-content][data-state='closed'][data-side='top'][data-last-action='failed']` | `border-bottom` | `var(--border-width-medium) solid var(--color-threat-element)` |
| 16 | `popover.css:253` | `[data-popover-content][data-state='closed'][data-side='bottom'][data-last-action='failed']` | `border-top` | `var(--border-width-medium) solid var(--color-threat-element)` |
| 17 | `popover.css:254` | `[data-popover-content][data-state='closed'][data-side='right'][data-last-action='failed']` | `border-left` | `var(--border-width-medium) solid var(--color-threat-element)` |
| 18 | `popover.css:255` | `[data-popover-content][data-state='closed'][data-side='left'][data-last-action='failed']` | `border-right` | `var(--border-width-medium) solid var(--color-threat-element)` |
| 19 | `popover.css:259` | `[data-popover-content][data-state='closed'][data-side='top'][data-last-action='cancelled'], [data-popover-content][data-state='closed'][data-side='top'][data-last-action='dismissed'], [data-popover-content][data-state='closed'][data-side='top'][data-last-action='dismissed-outside']` | `border-bottom` | `var(--border-width-medium) solid var(--color-neutral-element)` |
| 20 | `popover.css:262` | `[data-popover-content][data-state='closed'][data-side='bottom'][data-last-action='cancelled'], [data-popover-content][data-state='closed'][data-side='bottom'][data-last-action='dismissed'], [data-popover-content][data-state='closed'][data-side='bottom'][data-last-action='dismissed-outside']` | `border-top` | `var(--border-width-medium) solid var(--color-neutral-element)` |
| 21 | `popover.css:265` | `[data-popover-content][data-state='closed'][data-side='right'][data-last-action='cancelled'], [data-popover-content][data-state='closed'][data-side='right'][data-last-action='dismissed'], [data-popover-content][data-state='closed'][data-side='right'][data-last-action='dismissed-outside']` | `border-left` | `var(--border-width-medium) solid var(--color-neutral-element)` |
| 22 | `popover.css:268` | `[data-popover-content][data-state='closed'][data-side='left'][data-last-action='cancelled'], [data-popover-content][data-state='closed'][data-side='left'][data-last-action='dismissed'], [data-popover-content][data-state='closed'][data-side='left'][data-last-action='dismissed-outside']` | `border-right` | `var(--border-width-medium) solid var(--color-neutral-element)` |
### 1.2 A través de un privado (4)
| # | fichero:línea | selector | propiedad | valor |
| ---: | --- | --- | --- | --- |
| 1 | `popover.css:95` | `[data-popover-content]` | `inline-size` | `min(var(--_popover-width), var(--_popover-max-width), calc(100vw - var(--space-4)))` |
| 2 | `popover.css:96` | `[data-popover-content]` | `min-inline-size` | `var(--_popover-content-min-width-override, auto)` |
| 3 | `popover.css:98` | `[data-popover-content]` | `block-size` | `var(--_popover-content-height-override, auto)` |
| 4 | `popover.css:99` | `[data-popover-content]` | `min-block-size` | `var(--_popover-content-min-height-override, auto)` |
### 1.3 Literales (0)
_Ninguno._
## 2. Sistema transversal (3) — informativo, fuera del ratio
Un tema los alcanza **a nivel de sistema**, por diseño (recipe-contract §2).
| # | fichero:línea | selector | propiedad | valor |
| ---: | --- | --- | --- | --- |
| 1 | `popover.css:44` | `[data-popover-trigger]:not([data-archetype='field-trigger']):focus-visible` | `outline` | `var(--focus-ring-width) solid var(--focus-ring-color)` |
| 2 | `popover.css:208` | `[data-popover-arrow] polygon` | `fill` | `var(--depth-overlay-surface)` |
| 3 | `popover.css:213` | `[data-popover-arrow] path` | `stroke` | `var(--depth-overlay-border)` |
## 3. Privados de la receta — ¿de dónde sale su valor?
| privado | declaraciones | valor(es) | origen | ¿deriva de un público? |
| --- | ---: | --- | --- | :-: |
| `--_popover-padding-block` | 4 | `var(--popover-padding-block-md)`, `var(--popover-padding-block-sm)`, `var(--popover-padding-block-lg)`, `var(--popover-padding-block-xl)` | public | **sí** |
| `--_popover-padding-inline` | 4 | `var(--popover-padding-inline-md)`, `var(--popover-padding-inline-sm)`, `var(--popover-padding-inline-lg)`, `var(--popover-padding-inline-xl)` | public | **sí** |
| `--_popover-base-width` | 4 | `var(--popover-min-width-md)`, `var(--popover-min-width-sm)`, `var(--popover-min-width-lg)`, `var(--popover-min-width-xl)` | public | **sí** |
| `--_popover-width` | 1 | `var( --_popover-content-width-override, var(--_popover-match-anchor-width, var(--_popover-base-width)) )` | private | no |
| `--_popover-max-width` | 1 | `var(--_popover-content-max-width-override, var(--popover-max-width))` | public | **sí** |
| `--_popover-max-height` | 1 | `var( --_popover-content-max-height-override, var(--floating-available-height, calc(100dvh - var(--space-4))) )` | global | no |
Consumidos y **no declarados en el CSS** (vienen de `base.ts` o de un estilo inline del wrapper): `--_popover-content-height-override`, `--_popover-content-max-height-override`, `--_popover-content-max-width-override`, `--_popover-content-min-height-override`, `--_popover-content-min-width-override`, `--_popover-content-width-override`, `--_popover-match-anchor-width`.
## 4. Propuesta de corrección
- **Tiene eje `size`**: los tokens dimensionales van por talla (`{part}-{eje}-{k}`) apuntando al bundle `--size-{k}-*`, nunca al primitivo crudo (theming §5; el guard `recipe-css-contract` prohíbe el primitivo).
### 4.1 Tokens a declarar en `lib/recipes/base.ts` (9)
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
al final) y theming §6.7 (slots de color, modificador delante). Un token con
DOS valores distintos es una colisión de nombre: son dos knobs, o el nombre
no distingue lo que debería — se marca `⚠`.
| token (`--popover-…`) | scope TSC | valor propuesto | usos |
| --- | --- | --- | ---: |
| `trigger-padding-inline` | `root` | `var(--space-3)` | 1 |
| `trigger-radius` | `root` | `var(--radius-md)` | 1 |
| `trigger-bg` | `root` | `var(--color-surface-raised)` | 1 |
| `trigger-fg` | `root` | `var(--color-content-primary)` | 1 |
| `hover-trigger-bg` | `root` | `var(--color-surface-overlay)` | 1 |
| `hover-trigger-border` | `root` | `var(--color-border-strong)` | 1 |
| `open-trigger-bg` | `root` | `var(--color-surface-overlay)` | 1 |
| `open-trigger-border` | `root` | `var(--color-border-strong)` | 1 |
| `content-width` | `root` | `var( --_popover-content-width-override, var(--_popover-match-anchor-width, var(--_popover-base-width)) )` | 1 |
### 4.2 Sin nombre mecánico (17)
- **⚠ decisión: `height` es shorthand o eje físico — hay que partirlo en ejes lógicos antes de nombrarlo (recipe-contract §1, R-4.4)** — 1: `height`.
- **⚠ 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** — 3: `min-inline-size`, `block-size`, `min-block-size`.
- **⚠ decisión: `border-bottom` es shorthand o eje físico — hay que partirlo en ejes lógicos antes de nombrarlo (recipe-contract §1, R-4.4)** — 3: `border-bottom`.
- **⚠ decisión: `border-top` es shorthand o eje físico — hay que partirlo en ejes lógicos antes de nombrarlo (recipe-contract §1, R-4.4)** — 3: `border-top`.
- **⚠ decisión: `border-left` es shorthand o eje físico — hay que partirlo en ejes lógicos antes de nombrarlo (recipe-contract §1, R-4.4)** — 3: `border-left`.
- **⚠ decisión: `border-right` es shorthand o eje físico — hay que partirlo en ejes lógicos antes de nombrarlo (recipe-contract §1, R-4.4)** — 3: `border-right`.
### 4.3 Avisos sobre los tokens propuestos (1)
- **el privado `--_popover-width` debe pasar a leer este público (o desaparecer)** — `--popover-content-width`
### 4.4 Lo que hay que comprobar a mano (PLAN-theming §1.3 · §7.4)
- [ ] **Privado que no deriva de un público** — §3 lo marca; el privado debe leer el público o desaparecer.
- [ ] **Velo o acento en el nodo equivocado** (`archetype: 'item'` en un envoltorio, un `background` en shorthand que mata la capa de estado) — se mide desde el píxel hacia arriba.
- [ ] **Doble animación** al mover un sello a una superficie con animación propia — registro de `animationstart`/`animationend`.
- [ ] **Diff de computed = 0** en reposo · hover · abierto · disabled · foco, por talla, antes y después.
- [ ] **Centinela por token nuevo**: valor imposible en el root → el nodo lo sigue. Si no, el token miente.
## 5. Veredicto
<!-- veredicto:start -->
_(pendiente — lo escribe el autor; se conserva al regenerar)_
<!-- veredicto:end -->

Powered by TurnKey Linux.