VKUI icon indicating copy to clipboard operation
VKUI copied to clipboard

feat(Cell/SimpleCell/MiniInfoCell/RichCell): Use spacing size tokens

Open mendrew opened this issue 8 months ago • 5 comments

  • 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) для того чтобы всё привести к единой системе.

mendrew avatar Jun 19 '24 15:06 mendrew