fix(tradein/v2): audit #2081 residual — 404 short-circuit, disabled-tab tooltips, stale hex token, ДКП decode #2392

Merged

lekss361 merged 1 commit from fix/tradein-2081-audit-residual into main

2026-07-04 07:47:57 +00:00

lekss361 commented

2026-07-04 07:33:23 +00:00

Owner

Summary

3-й раунд UI/UX аудита /trade-in/v2 — почти всё уже закрыто предыдущими PR (#2086/#2091/#2093/#2264/#2265/#2273/#2311/#2332). Узкий финальный проход по остатку из последнего комментария (H2/M8-M12/L3-L7):

#	Итог
H2 (иерархия 3 цен)	confirmed-ok, код не тронут
M8/M9/M10/M11	confirmed-ok — уже корректны
L3 (404 доп.запросы)	fixed — sibling-хуки (analytics/location-coef/placement-history/sell-time) short-circuit'ят на подтверждённый 404 вместо лишних запросов
L5 (disabled-таб tooltip)	fixed — hover-title на неактивных пунктах user-меню
L7 (hardcode hex)	fixed — `#6f8195` → `tokens.muted` в TopNav
M12 (жаргон)	ДКП-расшифровка добавлена в HistoryView (тот же паттерн что ObjectSummary)
L4/L6	skipped — follow-up (mapper-level clamp / no contrast payoff)

Не трогал page.tsx mobile-layout (#2275, уже смержено) и HeroTransparency.tsx (#2377, параллельный PR).

Test plan

tsc --noEmit = 0 ошибок, next build = успех (агент)
code-review: ✅ APPROVE (1 minor — неточный комментарий, исправлен: расшифровка ДКП мирроринг не uppercase ResultPanel-стиля, а lowercase ObjectSummary hint-паттерна)
grep подтвердил zero overlap с #2275/#2377

Refs #2081

## Summary 3-й раунд UI/UX аудита `/trade-in/v2` — почти всё уже закрыто предыдущими PR (#2086/#2091/#2093/#2264/#2265/#2273/#2311/#2332). Узкий финальный проход по остатку из последнего комментария (H2/M8-M12/L3-L7): | # | Итог | |---|---| | H2 (иерархия 3 цен) | confirmed-ok, код не тронут | | M8/M9/M10/M11 | confirmed-ok — уже корректны | | **L3** (404 доп.запросы) | **fixed** — sibling-хуки (analytics/location-coef/placement-history/sell-time) short-circuit'ят на подтверждённый 404 вместо лишних запросов | | **L5** (disabled-таб tooltip) | **fixed** — hover-title на неактивных пунктах user-меню | | **L7** (hardcode hex) | **fixed** — `#6f8195` → `tokens.muted` в TopNav | | M12 (жаргон) | ДКП-расшифровка добавлена в HistoryView (тот же паттерн что ObjectSummary) | | L4/L6 | skipped — follow-up (mapper-level clamp / no contrast payoff) | Не трогал `page.tsx` mobile-layout (#2275, уже смержено) и `HeroTransparency.tsx` (#2377, параллельный PR). ## Test plan - [x] `tsc --noEmit` = 0 ошибок, `next build` = успех (агент) - [x] code-review: ✅ APPROVE (1 minor — неточный комментарий, исправлен: расшифровка ДКП мирроринг не uppercase ResultPanel-стиля, а lowercase ObjectSummary hint-паттерна) - [x] grep подтвердил zero overlap с #2275/#2377 Refs #2081

lekss361 added 1 commit 2026-07-04 07:33:24 +00:00

fix(tradein/v2): audit #2081 residual — 404 short-circuit, disabled-tab tooltips, stale hex token, ДКП decode

CI / openapi-codegen-check (pull_request) Has been skipped

Details

CI Trade-In / changes (pull_request) Successful in 8s

Details

CI / changes (pull_request) Successful in 8s

Details

CI Trade-In / backend-tests (pull_request) Has been skipped

Details

CI / backend-tests (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI Trade-In / frontend-checks (pull_request) Successful in 1m5s

Details

08cedad1bf

L3: sibling dashboard hooks (analytics/location-coef/placement-history/
sell-time) now short-circuit once restore-by-id confirms 404, instead
of each firing their own doomed request against the same dead
estimate id.
L5: dimmed non-interactive user-menu items (Профиль/Настройки/Помощь)
get a hover title explaining why they're disabled.
L7: TopNav logo subtitle used a stale literal #6f8195 predating the
C1 contrast sweep; now references tokens.muted.
M12: HistoryView's "ДКП-сделки" card gets the same one-time ДКП decode
convention ObjectSummary already uses.

H2/M8/M9/M10/M11 re-verified live/by-code — already correct, no
changes needed (see issue comment). L4 (HistoryView outlier clamp)
and L6 (blueprint-grid background-image) left as follow-ups
(mapper-level change / no contrast payoff respectively).

Refs #2081