Skip to content

size.menu: размеры подписей, иконка пункта и активные цвета из токенов - #112

Open
khaliulin wants to merge 2 commits into
mainfrom
size.menu
Open

size.menu: размеры подписей, иконка пункта и активные цвета из токенов#112
khaliulin wants to merge 2 commits into
mainfrom
size.menu

Conversation

@khaliulin

@khaliulin khaliulin commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Ветка от main. Компонент №11 — последний пункт плана перевода на новые токены и базу 16px.

Что сделано

Применены три токена, которые дизайн задал, а кит не использовал (их подсвечивал гейт G8):

  • menu.extend.iconSize — размер иконки пункта не задавался вовсе, иконка тянулась от размера шрифта; теперь 1.25rem, как в макете
  • menu.extend.extItem.caption.color — цвет подписи пункта был захардкожен семантикой color.fg.muted в обход токена компонента
  • menu.extend.extItem.icon.activeColor — иконка активного пункта красилась color.fg.inverse.default вместо токена компонента (fg.inverse.active); в тёмной теме это разные значения

Типографика пункта приведена к макету. Figma <Menu> (16005:21806) набирает пункт стилем secondary-font/text-base/font-normal/leading-none: fontSize.200 без интерлиньяжа. В ките стояли fontSize.300 (на ступень крупнее) и lineHeight.400 — заменены на fontSize.200 и lineHeight.none.

Заголовок группы приведён к макету. Стиль в Figma — primary-font/text-sm/font-bold/leading/uppercase: fontSize.100 без интерлиньяжа. В ките были fontSize.200 и lineHeight.400 — заменены. Вес оставлен из токена menu.submenuLabel.fontWeight (400) по решению на ревью, font-weight в CSS кита не задаётся.

Stories. У примеров снята подложка bg-surface-ground — компонент рисуется на фоне страницы, как в макете. В menu-custom иконки перестали быть цветными кружками: заливка и белый цвет убраны, иконка монохромная.

Проверки

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

Сверка с Figma (пересчёт базы 14 → 16): пункт Noto Sans 14px/1, заголовок группы TT Fellows 12px/1 bold uppercase #85888e, иконка 20px #2b2e33, padding панели 4px, фон белый, gap списка 4px, gap пункта 8px, радиус пункта 8px — совпадают. Тёмная тема: панель zinc.800, текст белый, заголовок группы zinc.400, иконки белые.

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

tokens.json не правил (экспорт из Figma):

  • вес заголовка группы. Текстовый стиль в макете привязывает fonts/fontWeight/bold (700), а токен компонента menu.submenuLabel.fontWeight = {fonts.fontWeight.regular}. По решению на ревью применяется токен (400); если верен стиль — правка в Figma.
  • радиус панели. menu.root.borderRadius = dimension.borderRadius.400 (0.875rem) против 10.5px в макете (0.75rem при базе 14).
  • hover пункта. У Aura для пункта меню есть только focus-токены; hover-фон кит задаёт семантикой color.bg.neutral.weak.hover. Если состояние нужно как токен компонента — его стоит завести в 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