Skip to content

Перевод select-button на новые токены и базу 16px - #100

Merged
ichiesov merged 1 commit into
size-updatefrom
size.select-button
Aug 21, 2026
Merged

Перевод select-button на новые токены и базу 16px#100
ichiesov merged 1 commit into
size-updatefrom
size.select-button

Conversation

@khaliulin

Copy link
Copy Markdown
Contributor

Ветка от size-update, процесс тот же, что в #97 и #98.

Главное: восстановлена размерная лестница

Кодмод из size-update («перевести стили на прямые пути токенов») увёл высоты select-button на ключи общей шкалы, которые существуют в обеих схемах, но означают разные ступени. Результат — lg и xlg стали ниже базового размера:

было @14 стало после кодмода стало сейчас
sm 28px 28px 32px
base 35px 40px 40px
lg 49px 26px 56px
xlg 56px 28px 64px

Источник правильных значений — токены togglebutton: select-button это контейнер кнопок-переключателей, и их размерная лестница задаётся дизайном именно там (extend.extSm/extLg/extXlg.iconOnlyWidth = 32/56/64px, extend.iconOnlyWidth = 40px).

Замеры в браузере

Корневой font-size = 16px, дробных пикселей нет. Лестница монотонна: 32 / 40 / 56 / 64.
Группа: радиус 12px, паддинг 4px, gap 4px. Кнопка внутри: радиус 8px.

Гейты

npm run check:component -- selectbutton — 0 FAIL, 5 WARN.

Все пять WARN — гейт G6 «ссылка на чужой компонент», и все пять осознанные: это и есть ссылки на togglebutton, обоснование записано в шапке файла. Одна из них (hoverBorderColor) была здесь и до миграции.

Не входит в ветку

Stories не тронуты намеренно. Их сейчас правит #96 — пересечение по src/stories/components/select-button/, конфликт был бы гарантирован.

В консоли stories есть ошибка NG0303: Can't bind to 'value' — история биндит [(value)], а у обёртки такого инпута нет. Это не следствие миграции: #96 как раз добавляет value и ControlValueAccessor. После его вливания уйдёт.

Сверка с Figma не проводилась — файл UI Kit (DS) v2.1 не был открыт в Figma Desktop.

Побочная находка для остальных веток

Тот же дефект «совпадающий ключ, другая ступень» кодмод оставил ещё в одном месте: date-picker.ts строки 40 и 183 используют dimension.size.1000 — было 84px при базе 14, стало 36px. Это −48px и заметно сломает выпадающий выбор года. Чинить в ветке datepicker.

@ichiesov
ichiesov merged commit baf3346 into size-update Aug 21, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants