-
Notifications
You must be signed in to change notification settings - Fork 58
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
Extend Spacing system #830
Conversation
According to VKCOM/VKUI#6684
To support the case when we want to keep size groups together in the token result name. For example: sizeS -> size-s sizeXs -> size-xs size2Xs -> size-2xs size2Xs3Regular2L -> size-2xs-regular-2l size2L3Regular-3S -> size-2l-3-regular-3s
Most of them are correct for us
Coverage report
Test suite run success406 tests passing in 37 suites. Report generated by 🧪jest coverage report action from 6789b72 |
Привет @8coon! Приемлимо ли это? |
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.
🔥 💅
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.
👍🏽
Перед отправкой этого реквеста на ревью убедитесь, что:
npm run build:local
),npm test
),npm run lint
), некоторые ошибки можно автоматически поправить с помощьюnpm run lint:fix
,ваш pull request содержит обновления документации,
согласовали их с дизайнерами. (c Алексеем Зайцевым)
Подробнее о внесении изменений в репозиторий токенов
Описание
Основано на задаче из VKUI: VKCOM/VKUI#6684 и обсуждении VKCOM/VKUI#5928. C дизайнерами VKUI также обсудили и именно этот набор токенов решено дальше поддерживать.
spacingSizeS
.2Xs
и3Xs
и2Xl
вместе, иначе со старой логикой получалось немного странно для таких токенов.Вот пример для spacingSize2Xl
Но сразу оговорюсь, что ничего не имею против оставить как есть, просто немного не интуитивно. По тестам проверил, что больше никакие токены не меняются, регулярка изменяет только такую запись.