Skip to content

fix(Dialog): скругление переведено на ступень шкалы - #72

Merged
fabulous-unicorn merged 1 commit into
release/2.0.0from
fix/dialog
Aug 13, 2026
Merged

fix(Dialog): скругление переведено на ступень шкалы#72
fabulous-unicorn merged 1 commit into
release/2.0.0from
fix/dialog

Conversation

@khaliulin

Copy link
Copy Markdown
Collaborator

Проблема

dialog.root.borderRadius ссылался на {dimension.borderRadius.max} = 1600 — то есть углы скруглялись по максимуму и упирались в размер окна.

В макете скругление задано ролью xl:

dialog/root/borderRadius → overlay/modal/borderRadius → primitive borderRadius/500 = 21

Решение

dialog.root.borderRadius переведён на {dimension.borderRadius.500} — ту же ступень шкалы, что и в макете. Значение: 20px.

Про 1px разницы

В нашей базе borderRadius.500 = 20, а не 21, и это не ошибка выгрузки, а следствие разных базовых шкал:

ступень Figma у нас
100 3.5 4
200 7 8
300 10.5 10
400 14 14
500 21 20

Макет считает от базы 14px (0.25 / 0.5 / 0.75 / 1 / 1.5rem), кит переведён на базу 16px. Кроме того, ровно 21 в нашей базе получить нечем: шкала sizing идёт шагом 0.125rem = 2px, нечётные значения в неё не попадают.

Приводить саму шкалу к макету в этом MR не стал — решение принято оставить borderRadius.500 как есть. {dimension.borderRadius.500} до этого MR не использовал никто (0 потребителей), так что ступень свободна и при желании её можно перенастроить отдельно.

Изменения

  • design-tokens/input/tokens.jsoncomponents.dialog.root.borderRadius: {dimension.borderRadius.max}{dimension.borderRadius.500}
  • src/theme/tokens/components/light.json, dark.json160020 (сгенерировано)
  • Dialog.test.tsx.snap — обновлён, дифф только по borderRadius

Проверки

  • yarn tokens:check — сгенерированные токены актуальны
  • yarn jest — 48 suites, 1708 тестов, всё зелёное (1 снапшот обновлён, дифф 160020)
  • prettier --check по затронутым файлам — чисто
  • dialog.root.borderRadius применяется только в DialogComponent.tsx:43

Не проверено

Визуальная проверка не проводилась — сверка велась по значениям токенов и переменным из Figma.

@khaliulin
khaliulin requested a review from a team as a code owner August 12, 2026 12:34
@khaliulin
khaliulin requested review from fabulous-unicorn and removed request for a team August 12, 2026 12:34
@Shoshanyaka Shoshanyaka added the release:v2.0.0 Release version v2.0.0 label Aug 12, 2026
@fabulous-unicorn
fabulous-unicorn merged commit 12adf04 into release/2.0.0 Aug 13, 2026
6 checks passed
@fabulous-unicorn
fabulous-unicorn deleted the fix/dialog branch August 13, 2026 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:v2.0.0 Release version v2.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants