Repository navigation
fix(Avatar): обводка переведена на inverse - #68
Merged
Merged
Conversation
fabulous-unicorn
requested review from
fabulous-unicorn
and removed request for
isokolovskii
August 12, 2026 13:12
fabulous-unicorn
approved these changes
Aug 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Проблема
Обводка
Avatarкрасилась серым —avatar.extend.borderColor→{color.border.neutral.strong}(light#cecfd2, dark#56595f). Обводка нужна для визуального разделения аватаров в группе, но серая линия читается как самостоятельный контур компонента, а не как разделитель.Решение
avatar.extend.borderColorв исходных токенах переведён на{color.border.neutral.inverse}— роль, совпадающую с цветом подложки:border.neutral.strong#cecfd2#56595fborder.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.tsxprettier --checkпо затронутым файлам — чистоavatar.extend.borderColorиспользуется только вAvatar.tsx:266Не проверено
Визуальная проверка в симуляторе не проводилась. В тёмной теме обводка получает
#2b2e33— это инверсная роль, а не буквально белый; она темнее фона поверхности (#404348), поэтому разделение аватаров сохраняется, но выглядит как тёмный зазор, а не как «вырез» по цвету подложки. Если для тёмной темы ожидается совпадение с фоном поверхности, роль нужно менять наbg.surface.default.default— скажите, поправлю.