fix(tradein/v2): источники N1+Restate вон, Домклик в текст; disabled-палитра до AA (#2081 follow-up) #2091

Merged
lekss361 merged 1 commit from fix/tradein-v2-audit-round2b into main 2026-06-28 23:03:49 +00:00
Owner

Follow-up к #2086 — два хвоста, найденные live-проверкой /trade-in/v2 под админом на реальной оценке (улица Академика Ландау 51/2, 9 аналогов).

M9 — корректировка списка источников

  • #2086 ошибочно убрал Домклик из текста «как считаем» (я доверился ссылке code-reviewer'а на #1968 «drop dead domklik/restate»). Live-данные показали: Домклик живой — даёт 3 из 9 аналогов (его оживил рескрейп #1846). Возвращён в текст.
  • N1.RU + RESTATE убраны из SOURCE_SLOTS (mappers.ts) — мёртвые, всегда «нет данных», по решению владельца не учитываем. Плитки источников и счётчик «N / 7» → «N / 5».
  • TOTAL_SOURCES = SOURCE_SLOTS.length (раньше хардкод 7) — единый источник истины, чтобы плитки/счётчик/текст не расходились.
  • Итоговый честный список: Циан, Я.Недвижимость, Авито, Домклик (+ Росреестр сделки).

C1 residual — disabled-палитра до WCAG AA

Аудит #2081 (C1) флагал «нет данных», но #2086 дотемнил только muted*-шкалу и пропустил disabled-палитру. Live computed-color sweep на результате поймал:

  • disabledValue («—» no-data) #c2cdd9 1.4:1#656b71 4.61:1
  • disabledLabel («нет данных» на неактивных плитках) #aab7c5 1.7:1#636b73 4.62:1

Верификация

tsc --noEmit = 0, next build = 0. После деплоя — повторный live-проход под админом (плитки=5, счётчик X/5, текст с Домклик, контраст «нет данных»/«—» ≥AA).

Self-merge при зелёном build.

Follow-up к #2086 — два хвоста, найденные **live-проверкой `/trade-in/v2` под админом** на реальной оценке (улица Академика Ландау 51/2, 9 аналогов). ### M9 — корректировка списка источников - #2086 ошибочно **убрал Домклик** из текста «как считаем» (я доверился ссылке code-reviewer'а на #1968 «drop dead domklik/restate»). Live-данные показали: **Домклик живой** — даёт 3 из 9 аналогов (его оживил рескрейп #1846). Возвращён в текст. - **N1.RU + RESTATE убраны** из `SOURCE_SLOTS` (`mappers.ts`) — мёртвые, всегда «нет данных», по решению владельца не учитываем. Плитки источников и счётчик **«N / 7» → «N / 5»**. - `TOTAL_SOURCES = SOURCE_SLOTS.length` (раньше хардкод `7`) — единый источник истины, чтобы плитки/счётчик/текст не расходились. - Итоговый честный список: **Циан, Я.Недвижимость, Авито, Домклик** (+ Росреестр сделки). ### C1 residual — disabled-палитра до WCAG AA Аудит #2081 (C1) флагал «нет данных», но #2086 дотемнил только `muted*`-шкалу и **пропустил disabled-палитру**. Live computed-color sweep на результате поймал: - `disabledValue` («—» no-data) `#c2cdd9` **1.4:1** → `#656b71` **4.61:1** - `disabledLabel` («нет данных» на неактивных плитках) `#aab7c5` **1.7:1** → `#636b73` **4.62:1** ### Верификация `tsc --noEmit` = 0, `next build` = 0. После деплоя — повторный live-проход под админом (плитки=5, счётчик X/5, текст с Домклик, контраст «нет данных»/«—» ≥AA). Self-merge при зелёном build.
lekss361 added 1 commit 2026-06-28 23:03:41 +00:00
fix(tradein/v2): источники N1+Restate вон, Домклик в текст; disabled-палитра до AA (#2081)
All checks were successful
CI / changes (pull_request) Successful in 6s
CI / backend-tests (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Has been skipped
c362b8592a
- M9 correction: #2086 ошибочно убрал Домклик из «как считаем» (по stale-ссылке на
  #1968 «drop dead domklik»), но live-проверка под админом показала Домклик живым
  (реально даёт лоты). Возвращён в текст. N1.RU + RESTATE убраны из SOURCE_SLOTS
  (мёртвые — всегда «нет данных», не учитываем) → плитки и счётчик «N / 7» → «N / 5».
  TOTAL_SOURCES = SOURCE_SLOTS.length (единый источник истины: плитки/счётчик/текст
  не расходятся). Итог: Циан, Я.Недвижимость, Авито, Домклик (+ Росреестр сделки).
- C1 residual: disabledValue («—» no-data) #c2cdd9 1.4:1 -> #656b71 4.61:1;
  disabledLabel («нет данных») #aab7c5 1.7:1 -> #636b73 4.62:1. Аудит #2081 флагал
  «нет данных», #2086 дотемнил только muted-шкалу — disabled-палитру пропустил.

tsc + next build green. Найдено live-проверкой /trade-in/v2 под админом (реальная оценка).
lekss361 added the
scope/frontend
tradein
ux
labels 2026-06-28 23:03:48 +00:00
lekss361 merged commit 4863e67f27 into main 2026-06-28 23:03:49 +00:00
lekss361 deleted branch fix/tradein-v2-audit-round2b 2026-06-28 23:03:49 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
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#2091
No description provided.