Skip to content

fix(theme): lift dark text-muted to gray.400 - #1122

Merged
robson-junior-azion merged 5 commits into
mainfrom
fix/dark-text-muted
Nov 3, 2026
Merged

robson-junior-azion merged 5 commits into
mainfrom
fix/dark-text-muted

Conversation

@gabriel-lisboa-azion

Copy link
Copy Markdown
Contributor

Summary

  • Dark --text-muted moves from gray.500 #808080 to gray.400 #999999. Muted copy then clears WCAG AA on every dark fill: canvas 7.37, surface 6.95, raised 6.47, selected 6.11, hover 5.58. At #808080, hover (4.03) and selected (4.41) failed.
  • --illustration-ink-muted follows it in dark, because it references the same value.

How to test

  1. Run pnpm storybook:dev and switch the toolbar theme to dark.
  2. Open any story with muted copy (e.g. Templates/Marketing/Network/NetworkSection, the "Built around your users" lead): the muted text reads #999999, and DevTools shows --text-muted: #999999 under [data-theme=dark].
  3. In light theme nothing changes: --text-muted stays #535353.

Notes

Dark --text-muted moves from #808080 to #999999 so muted copy clears AA on every dark fill, raised and hover surfaces included. Regenerating dist also carries the light illustration ground, line and ink-muted values that #1051 left stale.
# Conflicts:
#	apps/storybook/.storybook/test-visual/__image_snapshots__/linux/components-primitives-inputs-fieldselect--required--dark-mobile.png
#	apps/storybook/.storybook/test-visual/__image_snapshots__/linux/components-primitives-inputs-fieldselect--required--dark-tablet.png
#	apps/storybook/.storybook/test-visual/__image_snapshots__/linux/components-primitives-inputs-fieldselect--required.png
#	pnpm-workspace.yaml
@isaque-bock-azion
isaque-bock-azion dismissed stale reviews from guilherme-santana-azion and themself via d372333 October 31, 2026 00:04
@robson-junior-azion
robson-junior-azion merged commit e539d2a into main Nov 3, 2026
@robson-junior-azion
robson-junior-azion deleted the fix/dark-text-muted branch November 3, 2026 13:41
@github-actions github-actions Bot mentioned this pull request Nov 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants