VKUI
VKUI copied to clipboard
feat(Cell/SimpleCell/MiniInfoCell/RichCell): Use spacing size tokens
- related #6684
- [x] Unit-тесты
- [x] e2e-тесты
- [ ] Дизайн-ревью
Описание
Заменяем константы отступов на токены системы расстояний.
Токены системы расстояний из vkui-tokens: https://github.com/VKCOM/vkui-tokens/blob/2222b5b2208df9ea2d1f8d449e683f34696c6cd4/src/themeDescriptions/base/vk.ts#L730-L741
Вопросы
1.
Стоит ли нам заменять на токены отступы в режиме removable
такие как 44px
и 48px
?
https://github.com/VKCOM/VKUI/blob/f20e7904de7d43de14dc4e6026ab3eba0154ff8f/packages/vkui/src/components/Cell/Cell.module.css#L28-L29
https://github.com/VKCOM/VKUI/blob/f20e7904de7d43de14dc4e6026ab3eba0154ff8f/packages/vkui/src/components/Cell/Cell.module.css#L57
Так как у нас нет соответствующих токенов для таких размеров, то можно было бы использовать сложение имеющихся токенов, например
вместо 44px
писать calc(var(--vkui--spacing_size_3xl) + var(--vkui--spacing_size_4xl))
а вместо 48px
писать calc(var(--vkui--spacing_size_4xl) * 2)
Ответ: в данном случае оставим как есть, потому что тут не расстояние а размер иконки, и не хотелось бы, что это поехало при изменении токенов расстояний.
2.
У RichCell
есть нестандартный отступ у группы bottom
- 5px
. Оставим как есть или может быть в дизайне до 6px
увеличим?
https://github.com/VKCOM/VKUI/blob/991c6311c48970b6a90a56d0ed6e7ce87758ca81/packages/vkui/src/components/RichCell/RichCell.module.css#L100-L102
Ответ: решили использовать --vkui--spacing_size_s
(6px) для того чтобы всё привести к единой системе.