Skip to content

chore(sdds-sbcom): use theme tokens in component configs#2639

Draft
Yeti-or wants to merge 1 commit intodevfrom
yeti-or.packages/sdds-sbcom.improve-config
Draft

chore(sdds-sbcom): use theme tokens in component configs#2639
Yeti-or wants to merge 1 commit intodevfrom
yeti-or.packages/sdds-sbcom.improve-config

Conversation

@Yeti-or
Copy link
Copy Markdown
Contributor

@Yeti-or Yeti-or commented Mar 27, 2026

## SDDS-SBCOM

All

-   Замена использования css переменных на дизайн-токены из темы sdds-sbcom

What/why changed

Рефакторинг конфигов всех компонентов внутри библиотеки sdds-sbcom.
Используем токены из sdds-themes/tokens/sdds_sbcom вместо css переменных

Прежде чем перевести в статус "requested a review" убедитесь

  • Добавлен номер issue (если есть); — NO
  • Добавлены/обновлены cypress тесты если PR касается визуальной составляющей;

@github-actions
Copy link
Copy Markdown
Contributor

Theme Builder app deployed!

https://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-2639/

@github-actions
Copy link
Copy Markdown
Contributor

Documentation preview deployed!

website: https://plasma.sberdevices.ru/pr/pr-2639/
sdds-sbcom storybook: https://plasma.sberdevices.ru/pr/pr-2639/sdds-sbcom-storybook/

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Видимо h5 / h6 надо убирать ? сс @Yakutoc

h3,
h4,
h5,
h6,
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

нет таких токенов - h5 / h6

@@ -197,12 +223,12 @@ export const config = {
${tabsTokens.itemMarginLeft}: 1.5rem;
${tabsTokens.itemContentGap}: 0.625rem;

${tabsTokens.fontFamily}: var(--plasma-typo-h6-font-family);
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вот тут undefined, @neretin-trike это к вопросу что я пытался решить. Мы меняем sdds-com темы, и у нас потенциально компоненты разваливаются

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant