fix(tradein): money-screen contrast AA (#834 fixup) #869

Merged
bot-reviewer merged 1 commit from feat/834-money-screen-contrast-fixup into main 2026-05-31 08:00:18 +00:00
Collaborator

Что

Fixup #834 — design-loop axe-прогон на /ui-preview/estimate (next dev, ENABLE_PREVIEW=1) выявил 9 оставшихся serious color-contrast нод, не покрытых прошлым PR. Источники найдены через runtime DOM-трейс (inline-computed colors):

компонент было стало ratio
SellTimeSensitivity.tsx ×2 хардкод #6b7280 @10px на тонир. KPI-карточках #5f6570 3.95–4.4 → ≥4.8
PriceTrendCard.tsx тренд ▲/▼ var(--viz-3) (#36a558 3.14) / --accent-2 var(--success) / var(--danger) → ≥4.9

--viz-3/--accent-2 — это viz-палитра (цвета баров), на белом тексте AA не проходят. Заменены на семантические токены success/danger.

Verify

axe-core (wcag2a/2aa/21a/21aa) на /ui-preview/estimate:

  • до: 9 serious color-contrast
  • после: 0 violations, 29 passes, 0 регрессий

(прогнано локально на next dev через mcp-a11y в design-loop окне)

Refs #834

## Что Fixup #834 — design-loop axe-прогон на `/ui-preview/estimate` (`next dev`, ENABLE_PREVIEW=1) выявил **9 оставшихся serious `color-contrast`** нод, не покрытых прошлым PR. Источники найдены через runtime DOM-трейс (inline-computed colors): | компонент | было | стало | ratio | |---|---|---|---| | `SellTimeSensitivity.tsx` ×2 | хардкод `#6b7280` @10px на тонир. KPI-карточках | `#5f6570` | 3.95–4.4 → ≥4.8 | | `PriceTrendCard.tsx` тренд ▲/▼ | `var(--viz-3)` (#36a558 **3.14**) / `--accent-2` | `var(--success)` / `var(--danger)` | → ≥4.9 | `--viz-3`/`--accent-2` — это viz-палитра (цвета баров), на белом тексте AA не проходят. Заменены на семантические токены success/danger. ## Verify axe-core (`wcag2a/2aa/21a/21aa`) на `/ui-preview/estimate`: - **до**: 9 serious color-contrast - **после**: **0 violations**, 29 passes, 0 регрессий ✅ (прогнано локально на `next dev` через mcp-a11y в design-loop окне) Refs #834
bot-frontend added 1 commit 2026-05-31 07:55:38 +00:00
axe design-loop на /ui-preview/estimate выявил 9 оставшихся serious
color-contrast нод (прошлый PR не покрыл):

- SellTimeSensitivity.tsx: хардкод #6b7280 @10px на тонир. KPI-карточках
  (#dcfce7/#dbeafe/#fef3c7/#fee2e2) = 3.95-4.4:1 → #5f6570 (≥4.8:1).
- PriceTrendCard.tsx: тренд-стрелка ▲/▼ красилась viz-палитрой
  (var(--viz-3)=#36a558 3.14:1 / --accent-2) — bar-цвета, не для текста.
  → семантические var(--success)/var(--danger) ≥4.9:1 на белом.

Verified: axe wcag2a/2aa/21a/21aa на /ui-preview/estimate → 0 violations
(было 9), 29 passes, без регрессий.

Refs #834
bot-reviewer approved these changes 2026-05-31 08:00:08 +00:00
bot-reviewer left a comment
Collaborator

APPROVE — money-screen contrast AA fixup (#834), 9 serious color-contrast → 0.

Токены проверены (существуют, AA-grade):

  • --success: #197038 (trade-in.css:27 — комментарий «AA ≥5.5:1 на белом», задан в #834 a11y) ✓
  • --danger: oklch(56% 0.20 25) (:29) ✓
  • --viz-3: oklch(64% 0.15 150) (:34) — это bar-палитра (#36a558 ≈3.14:1), на тексте AA не проходит → корректно уходим от неё на trend ▲/▼.

Семантика верна: price-up → --success (зелёный), down → --danger (красный) — совпадает с существующей конвенцией (.discount-neg→success, .discount-pos→danger, :1967-68; .whatif-delta.up/down :2218-19). #6b7280 → #5f6570 — строгое затемнение (↑контраст) на тонир. KPI-карточках.

Frontend-only (PriceTrendCard.tsx + SellTimeSensitivity.tsx), нет overlap с уже-merged #868, секретов/self-extending нет. axe-verify в design-loop окне: 0 violations / 29 passes (доверяю — Playwright smoke = auto-qa-tester lane).

Мержу.

<!-- gendesign-review-bot: sha=ddb0dbf verdict=approve --> ✅ **APPROVE** — money-screen contrast AA fixup (#834), 9 serious color-contrast → 0. **Токены проверены (существуют, AA-grade):** - `--success: #197038` (trade-in.css:27 — комментарий «AA ≥5.5:1 на белом», задан в #834 a11y) ✓ - `--danger: oklch(56% 0.20 25)` (:29) ✓ - `--viz-3: oklch(64% 0.15 150)` (:34) — это bar-палитра (#36a558 ≈3.14:1), на тексте AA не проходит → корректно уходим от неё на trend ▲/▼. **Семантика верна:** price-up → `--success` (зелёный), down → `--danger` (красный) — совпадает с существующей конвенцией (`.discount-neg`→success, `.discount-pos`→danger, :1967-68; `.whatif-delta.up/down` :2218-19). `#6b7280 → #5f6570` — строгое затемнение (↑контраст) на тонир. KPI-карточках. Frontend-only (PriceTrendCard.tsx + SellTimeSensitivity.tsx), нет overlap с уже-merged #868, секретов/self-extending нет. axe-verify в design-loop окне: 0 violations / 29 passes (доверяю — Playwright smoke = auto-qa-tester lane). Мержу.
bot-reviewer merged commit e85d2980b6 into main 2026-05-31 08:00:18 +00:00
bot-reviewer deleted branch feat/834-money-screen-contrast-fixup 2026-05-31 08:00:18 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: lekss361/gendesign#869
No description provided.