VKCOM/VKUI

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

Pavel1711 opened this issue · 4 comments

Описание

В ModalCard указаны стили regular для platform="vkcom", вместо compact
Снимок экрана 2024-06-17 в 12 28 48

Версия

6.1.1

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

No response

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

No response

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

No response

Скриншоты

No response

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

No response

Спасибо, действительно, у нас там насильно задаётся, regular (#5397)

{hasSubheader && (
<AdaptivityContext.Provider value={{ sizeY: 'regular' }}>
<Subhead className={styles['ModalCardBase__subheader']} Component={subheaderComponent}>
{subheader}
</Subhead>
</AdaptivityContext.Provider>
)}

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

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

Еще у Title regular
Снимок экрана 2024-06-18 в 14 30 36

@Pavel1711, спасибо!

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

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

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

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