Skip to content

size.chip: размер лейбла и иконки, граница без слоя синонимов - #115

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

size.chip: размер лейбла и иконки, граница без слоя синонимов#115
khaliulin wants to merge 1 commit into
mainfrom
size.chip

Conversation

@khaliulin

Copy link
Copy Markdown
Contributor

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

Что сделано

Снят блок :root — две переменные --p-chip-extend-* и два чтения их сырым var(). Граница чипа теперь берётся через dt('chip.extend.borderWidth') и dt('chip.extend.borderColor').

Типографика лейбла приведена к макету. Figma <Chip> (312:1298) набирает текст стилем secondary-font/text-base/font-normal/leading-tight: fontSize.200 без интерлиньяжа. В ките стояли fontSize.300 (на ступень крупнее) и lineHeight.400 — то же расхождение, что нашлось в menu.

Кольцо фокуса иконки удаления — на токене компонента. Было dt('color.border.focus') напрямую, хотя у компонента есть chip.removeIcon.focusRing.color (он и ссылается на этот же семантический токен). Теперь CSS зависит от токена компонента, как задумано в экспорте.

Иконке чипа задан размер. Aura читает его из chip.icon.font.size, а в экспорте есть только chip.icon.size — переменная --p-chip-icon-font-size не объявлена, и глиф .ti рисовался размером шрифта родителя (14px) внутри бокса 16px. Теперь font-size берётся из chip.icon.size, глиф и бокс совпали.

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

Проверки

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

Сверка с Figma (пересчёт базы 14 → 16): gap 8px, padding 8×4px, радиус 8px, фон #e2e2e4, цвет текста и иконок #2b2e33, прозрачная граница, кольцо фокуса 4px, disabled 50% — совпадают.

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

Расхождений значений с макетом нет — tokens.json по чипу совпадает с Figma. Единственный пробел — отсутствующий chip.icon.fontSize, которого Aura ждёт отдельно от chip.icon.size; сейчас закрыт в CSS кита размером иконки.

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