VKCOM / VKUI

VKUI – это набор React-компонентов, с помощью которых можно создавать интерфейсы, внешне неотличимые от наших iOS и Android приложений.
https://vkcom.github.io/VKUI/
MIT License
1.03k stars 186 forks source link

[Bug]: Устанавливаются стили regular в ModalCard для vkcom #7027

Open Pavel1711 opened 4 months ago

Pavel1711 commented 4 months ago

Описание

В ModalCard указаны стили regular для platform="vkcom", вместо compact

Снимок экрана 2024-06-17 в 12 28 48

Версия

6.1.1

В каких браузерах воспроизводится проблема?

No response

Шаги воспроизведения

No response

Ожидаемое поведение

No response

Скриншоты

No response

Пример с воспроизведением

No response

mendrew commented 4 months ago

Спасибо, действительно, у нас там насильно задаётся, regular (https://github.com/VKCOM/VKUI/pull/5397) https://github.com/VKCOM/VKUI/blob/9172a17539cab522ac520334a2e76c527591040c/packages/vkui/src/components/ModalCardBase/ModalCardBase.tsx#L148-L154

Хотя для темы VKCOM subhead должен быть compact.

@Pavel1711 , eсть ещё что-нибудь кроме subhead, что выбивается из дизайн-системы?

Pavel1711 commented 4 months ago

Еще у Title regular

Снимок экрана 2024-06-18 в 14 30 36
mendrew commented 4 months ago

@Pavel1711, спасибо!

компоненты VKUI следуют дизайн-системе и именно там задаются основные правила, в том числе и для типографики, в зависимости от sizeClass (regular и compact).

В данный момент для Title в VKUI всё задано верно. И для regular, и для compact в дизайн-системе используются regular токены типографики. Такое бывает и это нормально.

А вот с subhead не всё так однозначно, выясняю.

mendrew commented 4 months ago

С subhead пришли к выводу, что надо оставить regular для всех платформ. И добавить условие, чтобы для desktop использовать токен --vkui--font_text--font_size--compact.