Skip to content

refactor(design): Update color tokens blue--700 and text--secondary#3060

Merged
julia-wjw merged 5 commits intomasterfrom
JOB-156764/update-token-text-secondary
Apr 16, 2026
Merged

refactor(design): Update color tokens blue--700 and text--secondary#3060
julia-wjw merged 5 commits intomasterfrom
JOB-156764/update-token-text-secondary

Conversation

@julia-wjw
Copy link
Copy Markdown
Contributor

@julia-wjw julia-wjw commented Apr 13, 2026

Why Is This Changing?

  • text--secondary does not pass contrast standards against some backgrounds.

What Is Changing?

  • Updated color.base.blue-.700 to hsl(197, 21%, 36%) (equivalent to #49646F)
  • Point color.text-.secondary at {color.base.blue-.700} for light theme semantic tokens

Consumer Impact

  • Impact: Low
  • Action required: None
  • Breaking change: No

Validation

  • Visual review of secondary text in Storybook/site if applicable

Made with Cursor

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Apr 13, 2026

Deploying atlantis with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2d2c1a2
Status: ✅  Deploy successful!
Preview URL: https://f7120d2d.atlantis.pages.dev
Branch Preview URL: https://job-156764-update-token-text.atlantis.pages.dev

View logs

@julia-wjw julia-wjw marked this pull request as draft April 13, 2026 16:23
- Added modulePathIgnorePatterns to avoid duplicate haste entries in jest configurations for components-native.
- Updated TypeScript configuration to include specific file patterns and exclude mock directories.
- Adjusted color values in visual regression snapshots for consistency across components.
Copy link
Copy Markdown
Contributor

@jdeichert jdeichert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey! Just did a quick review and caught some things the LLM probably shouldn't have touched. Unclear why the LLM decided to do that.. I'd have to dig in but don't have time right now.

See comments, reverting those files will unblock this PR 👍

Comment thread jest.config.js Outdated
Comment thread packages/components-native/tsconfig.json Outdated
Comment thread packages/components-native/jest.config.js Outdated
@julia-wjw julia-wjw requested a review from jdeichert April 16, 2026 14:03
@julia-wjw julia-wjw marked this pull request as ready for review April 16, 2026 14:06
Copy link
Copy Markdown
Contributor

@jdeichert jdeichert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks for updating all the screenshots!

@julia-wjw julia-wjw merged commit 367b53f into master Apr 16, 2026
14 checks passed
@julia-wjw julia-wjw deleted the JOB-156764/update-token-text-secondary branch April 16, 2026 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants