fix(tradein/ui): не обрезать длинные названия городов в селекторе (#2576)
All checks were successful
CI Trade-In / changes (pull_request) Successful in 9s
CI / changes (pull_request) Successful in 10s
CI Trade-In / backend-tests (pull_request) Has been skipped
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
CI Trade-In / frontend-checks (pull_request) Successful in 1m8s
All checks were successful
CI Trade-In / changes (pull_request) Successful in 9s
CI / changes (pull_request) Successful in 10s
CI Trade-In / backend-tests (pull_request) Has been skipped
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
CI Trade-In / frontend-checks (pull_request) Successful in 1m8s
Живая проверка прода после #2584: триггер городского дропдауна (pp-dd-trigger-dashed, ParamsPanel.tsx) — фиксированные 176x22px, font-size 11px. Дефолтный лейбл "Определить автоматически" (~146px в Manrope 400, замерено opentype.js против реального шрифта прода) не влезал в однострочный бюджет ~143px, переносился на вторую строку и обрезался высотой триггера. Заменил UNCONFIRMED_CITY_LABEL на "Автоопределение" (~96px, большой запас) — сохраняет смысл, перекликается с "Авто" у РАДИУС АНАЛИЗА, но не двусмысленно рядом с названиями городов. Самое длинное название города в CITY_LABELS, "Каменск-Уральский" (~108px), укладывается в тот же бюджет без переноса — второго фикса не требует.
This commit is contained in:
parent
a7fbd348e0
commit
62aca75994
1 changed files with 13 additions and 1 deletions
|
|
@ -64,8 +64,20 @@ export const DEFAULT_CITY: OblastCity = OBLAST_CITIES[0];
|
|||
* выбран/определён). Честнее статичного "Екатеринбург": не выдаёт наш
|
||||
* best-guess за подтверждённый выбор пользователя (deep-review R2 —
|
||||
* `city_hint` не должен отправляться в этом состоянии).
|
||||
*
|
||||
* Fix (эпик #2576, живая проверка прода после #2584): дропдаун-триггер в
|
||||
* ParamsPanel.tsx — фиксированные 176×22px, font-size 11px. Прежний текст
|
||||
* "Определить автоматически" (~146px в Manrope 400 — измерено
|
||||
* opentype.js-сравнением с рендером на проде) не влезал в однострочный
|
||||
* бюджет ~143px (176 - padding 2×13 - место под "▼"), переносился на вторую
|
||||
* строку и обрезался высотой триггера. "Автоопределение" (~96px) укладывается
|
||||
* с большим запасом, сохраняя смысл — сравни с "Авто" у РАДИУС АНАЛИЗА
|
||||
* (тот же проектный паттерн для автоматического режима), но без риска
|
||||
* двусмысленности рядом с названиями городов. Самое длинное название города
|
||||
* в CITY_LABELS, "Каменск-Уральский" (~108px), укладывается в тот же бюджет
|
||||
* без переноса — проверено тем же замером, отдельного фикса не требует.
|
||||
*/
|
||||
export const UNCONFIRMED_CITY_LABEL = "Определить автоматически";
|
||||
export const UNCONFIRMED_CITY_LABEL = "Автоопределение";
|
||||
|
||||
/** RU-лейблы для дропдауна — тот же формат (string[]), что dropdownOptions.* в v2/fixtures.ts. */
|
||||
export const CITY_LABELS: string[] = OBLAST_CITIES.map((c) => c.label);
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue