Skip to content

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

Open
khaliulin wants to merge 2 commits into
size-updatefrom
size.button
Open

Перевод button на новые токены и базу 16px#97
khaliulin wants to merge 2 commits into
size-updatefrom
size.button

Conversation

@khaliulin

Copy link
Copy Markdown
Contributor

Первый компонент по плану переезда на базу 16px. Ветка от size-update.

Что сделано

button.ts: 322 → 176 строк.

  • удалён export default — 150 строк мёртвого дубля, который никто не импортировал (map-tokens.ts берёт именованный buttonCss);
  • удалён блок :root со слоем синонимов --p-button-extend-*: PrimeUIX сам выпускает переменную для каждого токена пресета, отбрасывая сегменты root/extend (button.extend.extLg.height--p-button-ext-lg-height);
  • все 33 сырых var(--p-*) заменены на dt() — три из них были мертвы после смены схемы токенов (--p-opacity-1000, --p-controls-icon-only-850/900);
  • применены токены, которые дизайн задаёт, а кит не использовал: толщина рамки, gap у xlg (перекрывался правилом для lg), фон link-кнопки, фон в фокусе для severity в вариантах outlined и text;
  • добавлены высоты для base и sm — новые токены их задают, CSS не применял.

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

Корневой font-size = 16px, дробных пикселей нет.

height padding X/Y radius font
sm 28px 14/8 14px 14px
base 36px 14/8 14px 16px
lg 48px 24/14 14px 20px
xlg 56px 24/20 14px 20px

Гейты

npm run check:component -- button — без FAIL и WARN.
npx prettier --check — чисто. Консоль на всех stories — 0 ошибок.

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

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

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.

1 participant