Skip to content

fix(Avatar): обводка переведена на inverse - #68

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

fabulous-unicorn merged 1 commit into
release/2.0.0from
fix/avatar

Conversation

@khaliulin

@khaliulin khaliulin commented Aug 12, 2026 •

Copy link
Copy Markdown
Collaborator

Проблема

Обводка Avatar красилась серым — avatar.extend.borderColor → {color.border.neutral.strong} (light #cecfd2, dark #56595f). Обводка нужна для визуального разделения аватаров в группе, но серая линия читается как самостоятельный контур компонента, а не как разделитель.

Решение

avatar.extend.borderColor в исходных токенах переведён на {color.border.neutral.inverse} — роль, совпадающую с цветом подложки:

light dark
было — border.neutral.strong #cecfd2 #56595f
стало — border.neutral.inverse #ffffff #2b2e33

Именно эта роль уже назначена дизайном на avatar.group.borderColor — то есть цвет разделителя в группе зафиксирован в токенах, просто сам компонент брал другой. Теперь extend.borderColor и group.borderColor совпадают.

borderWidth не менялся — остаётся dimension.borderWidth.100 (1px).

Изменения

  • design-tokens/input/tokens.json — components.avatar.extend.borderColor: {color.border.neutral.strong} → {color.border.neutral.inverse}
  • src/theme/tokens/components/light.json — #cecfd2 → #ffffff (сгенерировано)
  • src/theme/tokens/components/dark.json — #56595f → #2b2e33 (сгенерировано)
  • Avatar.test.tsx.snap — обновлён, дифф только по borderColor

Итоговый дифф ветки к базе — три строки токенов и снапшот. Промежуточные коммиты ветки содержат откатанный вариант с transparent и нулевой рамкой; финальное состояние их не включает.

Проверки

  • yarn tokens:check — сгенерированные токены актуальны
  • yarn jest — 48 suites, 1708 тестов, 1567 снапшотов, всё зелёное
  • tsc --noEmit — чисто; eslint — только предсуществующие warning'и в Dialog.tsx
  • prettier --check по затронутым файлам — чисто
  • avatar.extend.borderColor используется только в Avatar.tsx:266

Не проверено

Визуальная проверка в симуляторе не проводилась. В тёмной теме обводка получает #2b2e33 — это инверсная роль, а не буквально белый; она темнее фона поверхности (#404348), поэтому разделение аватаров сохраняется, но выглядит как тёмный зазор, а не как «вырез» по цвету подложки. Если для тёмной темы ожидается совпадение с фоном поверхности, роль нужно менять на bg.surface.default.default — скажите, поправлю.

@khaliulin
khaliulin requested a review from a team as a code owner August 12, 2026 10:41
@khaliulin
khaliulin requested review from isokolovskii and removed request for a team August 12, 2026 10:41
@Shoshanyaka Shoshanyaka added the release:v2.0.0 Release version v2.0.0 label Aug 12, 2026
@khaliulin khaliulin changed the title fix(Avatar): обводка контейнера убрана fix(Avatar): обводка переведена на inverse Aug 12, 2026
@fabulous-unicorn
fabulous-unicorn merged commit 8b1841e into release/2.0.0 Aug 12, 2026
4 checks passed
@fabulous-unicorn
fabulous-unicorn deleted the fix/avatar branch August 12, 2026 13:20
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