Skip to content

size.badge: акцентный шрифт и dot-варианты без слоя синонимов - #114

Open
khaliulin wants to merge 1 commit into
mainfrom
size.badge
Open

size.badge: акцентный шрифт и dot-варианты без слоя синонимов#114
khaliulin wants to merge 1 commit into
mainfrom
size.badge

Conversation

@khaliulin

Copy link
Copy Markdown
Contributor

Ветка от main. Второй компонент из группы «слой синонимов + сырые var()» после avatar.

Что сделано

Снят блок :root — семь переменных --p-badge-extend-*, которые тема выпускает сама, и семь чтений их сырым var(). Значения dot-варианта (цвета severity, размеры lg/xlg, обнуление padding) теперь берутся через dt() из badge.extend.*.

Бейдж получил акцентный шрифт. Figma <Badge> (21:1193) набирает значение стилем primary-font/text-sm/font-normal: fontFamily.heading (TT Fellows), fontSize.100, fontWeight.regular, lineHeight.350. Размер и вес приходят токенами badge.root.*, но font-family у Aura для бейджа нет вовсе — до этой ветки бейдж рисовался дефолтным стеком ui-sans-serif, system-ui…. Добавлены font-family и line-height; тот же пропуск чинили в tag.

Stories. У трёх примеров снята подложка bg-surface-ground, отступ p-4 сохранён.

Проверки

  • check-component badge — было 8 FAIL (7×G3 сырые var, G5 слой синонимов), стало 0 FAIL, 0 WARN
  • check-stories-bg badge — подложек нет
  • check-dt-refs — мёртвых ссылок в badge.ts нет
  • рантайм: база 16px, дробных пикселей нет, консоль без ошибок, из необъявленных переменных — только --p-scrollbar-width
  • prettier --check по своим файлам чист
  • границы ветки: tokens/components/badge.ts и stories/components/badge/examples/

Сверка с Figma (пересчёт базы 14 → 16): высота и min-width 24px, padding 8px, радиус 8px, lg 28px, xl 32px, точка 8px, точка lg 12px, точка xlg 16px, шрифт TT Fellows 12px/18px, цвета severity (#aafbb7 фон обычного success, #f47f77 точка danger) — совпадают.

Вопросы к дизайну

Расхождений значений с макетом не нашёл — tokens.json по бейджу совпадает с Figma один в один.

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