-
Notifications
You must be signed in to change notification settings - Fork 72
[LG-5748] chore: Add Text and Border Color Variants #3321
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: c481f33 The changes in this PR will be included in the next version bump. This PR includes changesets to release 87 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Size Change: +1.81 kB (+0.1%) Total Size: 1.79 MB
ℹ️ View Unchanged
|
|
Coverage after merging ar/LG-5748-color-tokens into main will be
Coverage Report
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR enhances the tokens package by adding new color variants and updating existing ones to better support the design system's needs. The changes provide more granular control over success states and tertiary UI elements.
Key changes:
- Added
Tertiaryborder color variant for both light and dark modes - Added
SuccessandSuccessLinktext color variants for both light and dark modes - Updated
SuccessandErrorborder color values to use lighter/darker shades for better visual hierarchy - Enhanced Storybook table readability by capitalizing variant type labels
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| packages/tokens/src/color/color.types.ts | Added new variant enum values: Tertiary and SuccessLink |
| packages/tokens/src/color/lightModeColors.ts | Added Tertiary border colors, Success and SuccessLink text colors, and updated existing Success and Error color shades |
| packages/tokens/src/color/darkModeColors.ts | Added Tertiary border colors, Success and SuccessLink text colors, and updated existing Success and Error color shades |
| packages/tokens/src/Tokens.stories.tsx | Added CSS text-transform capitalization to improve variant label readability in Storybook |
| .changeset/moody-clouds-lead.md | Documented changes as a minor version bump with description of new variants |
✍️ Proposed changes
This PR adds new color token variants to support additional design system needs. The changes include:
tertiaryvariant for border colors (both light and dark modes)successvariant for text colors (both light and dark modes)successLinkvariant for text colors (both light and dark modes)successanderrorborder color values to use more appropriate shadesThese token additions provide more granular control over color variants across the design system, particularly for success states and tertiary UI elements.
🎟️ Jira ticket: LG-5748
✅ Checklist
pnpm changesetand documented my changes🧪 How to test changes
tertiary,success,successLink) are displayed correctly in both light and dark modessuccessanderrorvariants render with the updated color valuessuccessandsuccessLinkvariants display appropriately across all interaction states