-
Notifications
You must be signed in to change notification settings - Fork 55
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
Change spacing system tokens #859
Conversation
According to conversation with design
Coverage report
Test suite run success406 tests passing in 37 suites. Report generated by 🧪jest coverage report action from 9c597b9 |
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.
Всё круто, спасибо.
Думаю, мы можем избавлять 3XS, вряд ли его успеют заметить и использовать, учитывая что в дизайне его нет.
Спасибо за ревью! Согласен, но если теперь удалим токен, то придётся выпускать мажорную версию vkui-tokens с |
Жестоко |
У мобильных разрабов слетели значения токенов, и надо бы поскорее вмержить этот реквест, чтобы починить всё обратно) |
Перед отправкой этого реквеста на ревью убедитесь, что:
npm run build:local
),npm test
),npm run lint
), некоторые ошибки можно автоматически поправить с помощьюnpm run lint:fix
,ваш pull request содержит обновления документации,
согласовали их с дизайнерами.
Подробнее о внесении изменений в репозиторий токенов
Описание
В первом PR #830 мы немного немного разошлись с ожиданиями дизайна, поэтому перетряхиваем значение токенов из системы расстояний.
Не удаляю, но помечаю как
deprecated
spacingSize3XS
так как мы его пока что не собираемся использовать. Лучше бы, конечно, удалить совсем, если вы не против.