Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ветка от
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нет--p-scrollbar-widthprettier --checkчистcheck-stories-bg menu— подложек нетtokens/components/menu.tsиstories/components/menu/examples/Сверка с Figma (пересчёт базы 14 → 16): пункт
Noto Sans14px/1, заголовок группыTT Fellows12px/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).focus-токены; hover-фон кит задаёт семантикойcolor.bg.neutral.weak.hover. Если состояние нужно как токен компонента — его стоит завести в Figma.