Skip to content

fix(Slider): цвет лейблов в тёмной теме - #71

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

fix(Slider): цвет лейблов в тёмной теме#71
fabulous-unicorn merged 1 commit into
release/2.0.0from
fix/slider

Conversation

@khaliulin

@khaliulin khaliulin commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Проблема

Лейблы со значениями у Slider в тёмной теме нечитаемы: тёмный текст на тёмном фоне.

Они выводились голым <Text> из react-native без стилей, а у него цвет по умолчанию чёрный, независимо от темы.

<Text>{positionMinRange.toFixed(0)}</Text>

Где именно

Лейблы живут не в компоненте, а в его сторибуке — Slider.stories.tsx. Сам Slider.tsx текста не содержит вовсе: у него нет ни пропа лейбла, ни единого <Text>. Макет это подтверждает — компонент-сет <Slider> в Figma (383:1844) состоит из 6 вариантов state x range и не содержит текстовых слоёв.

То есть подписи значений — часть демонстрации, а не публичного API компонента, и правка ограничена сторибуком.

Решение

Лейблам задан цвет из семантической роли fg.default, разметка не менялась:

const styles = StyleSheet.create(({ semantic }) => ({
  label: { color: semantic.colorScheme.color.fg.default },
}))
light dark
fg.default #2b2e33 #ffffff

Правка затрагивает только цвет. Размер шрифта, начертание, межстрочный интервал и семейство остаются дефолтными для <Text> — типографика лейблов не менялась.

Изменения

  • src/components/Slider/Slider.stories.tsx — трём <Text> добавлен style={styles.label}, заведён стиль label

Проверки

  • yarn jest — 48 suites, 1708 тестов, всё зелёное (снапшоты не затронуты — сторибуки не покрыты тестами)
  • tsc --noEmit — чисто
  • eslint — 2 warning'а, оба предсуществующие в Checkbox.tsx и Dialog.tsx
  • prettier --check — чисто

Замечено рядом

Та же проблема есть в Dialog.stories.tsx, DialogComponent.stories.tsx и Tabs.stories.tsx — там тоже голый <Text> без цвета. Вне рамок этого MR.

Не проверено

Визуальная проверка сторибука в тёмной теме не проводилась.

@khaliulin
khaliulin requested a review from a team as a code owner August 12, 2026 12:09
@khaliulin
khaliulin requested review from isokolovskii and removed request for a team August 12, 2026 12:09
@Shoshanyaka Shoshanyaka added the release:v2.0.0 Release version v2.0.0 label Aug 12, 2026
@fabulous-unicorn
fabulous-unicorn merged commit 9746005 into release/2.0.0 Aug 12, 2026
6 checks passed
@fabulous-unicorn
fabulous-unicorn deleted the fix/slider branch August 12, 2026 13:30
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