-
Notifications
You must be signed in to change notification settings - Fork 185
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(Typography): Support fontWeightBase tokens #7549
feat(Typography): Support fontWeightBase tokens #7549
Conversation
Add accent prop to specify base/accent font weight mode By default accent is enabled
That we change the weight tokens
size-limit report 📦
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
e2e tests |
👀 Docs deployed
Commit 369ed67 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #7549 +/- ##
=======================================
Coverage 95.14% 95.14%
=======================================
Files 384 384
Lines 11221 11226 +5
Branches 3685 3688 +3
=======================================
+ Hits 10676 10681 +5
Misses 545 545
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💅
✅ v6.7.0 🎉 |
Описание
font-weight
в дизайн системе может выглядеть как отдельный режим.Мы позволяем менять
font-weight
с помощью свойстваweight
компонентаTypography
, который под капотом у всех других типографических элементов.VKUI/packages/vkui/src/components/Typography/Typography.tsx
Lines 21 to 24 in 716657b
Но Typography поддерживает не все токены, которые могут быть использованы в дизайн-системе.
Вот все токены
fontWeight
пока что существующие в vkui-tokens. Ссылка на токены темыparadigm
https://github.com/VKCOM/VKUI/blob/716657bed3a0498ac2458ec0b53f9e3bfa46da0d/packages/vkui/src/components/Typography/Typography.module.css#L12-L27
Сейчас VKUI поддерживает только разные уровни
fontWeightAccent
VKUI/packages/vkui/src/components/Typography/Typography.module.css
Lines 12 to 27 in 716657b
Стоит также поддержать и уровни
fontWeightBase
, я видел использование таких токенов в дизайн системе, независимо от regular/compact. Например жирность была fontWeightBase равная 700 и в режиме compact и в режиме regular, что нельзя сейчас поддержать в VKUI.Надо придумать как эти токены добавить.
Изменения
Добавлено свойство
useAccentWeight
, которое по умолчаниюtrue
, так как в VKUI по умолчанию используются токеныfontWeightAccent
. Сейчас, чтобы использовались токеныfontWeightBase
надо явно задатьuseAccentWeight={false}
.Явно указывать
accent
по умолчанию какfalse
выглядит как breaking change, так как могут быть пользователиText
компонента, которые ожидают уже, что там будетaccent
токены по умолчанию.Добавил TODO поменять
accent
наfalse
в V7.Добавлять
useBaseWeight
, вместоuseAccentWeight
не стал, выглядит не плохо с одной стороны, ноbase
, это как бы база, аaccent
уже как бы отличный вариант отbase
, надстройка, хочется её такой и оставить.В общем, я больше склоняюсь к
useAccentWeight
.Release notes
Улучшения
useAccentWeight
для использованияfontWeightAccent
токенов, если требуется поменять начертание текста с помощью свойстваweight
. По умолчанию в VKUIuseAccentWeight={true}
, то есть при исползовании свойстваweight
применяются токеныfontWeightAccent
. Если нужно, чтобы использовались токеныfontWeightBase
необходимо явно указатьuseAccentWeight={false}
. В v7useAccentWeight
по умолчанию будетfalse
.