Skip to content
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

BadgeCount: update the color tokens to be more accurate token for the usage #2392

Merged
merged 2 commits into from
Sep 6, 2024

Conversation

shleewhite
Copy link
Contributor

@shleewhite shleewhite commented Sep 5, 2024

📌 Summary

If merged, this PR would change the usage of foreground colors in the BadgeCount component as background and border colors to just the palette colors.

📸 Screenshots

Screenshot 2024-09-05 at 12 43 58 PM
Screenshot 2024-09-05 at 12 44 12 PM

🔗 External links

Jira ticket: HDS-3817
Figma file: https://www.figma.com/design/iweq3r2Pi8xiJfD9e6lOhF/%5BWIP%5D-HDS-Components-v2.0?node-id=205-564&t=16lLFxX1rGve3Zqp-1


👀 Component checklist

💬 Please consider using conventional comments when reviewing this PR.

Copy link

vercel bot commented Sep 5, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
hds-showcase ✅ Ready (Inspect) Visit Preview Sep 6, 2024 1:11pm
hds-website ✅ Ready (Inspect) Visit Preview Sep 6, 2024 1:11pm

@shleewhite shleewhite marked this pull request as ready for review September 5, 2024 17:15
@shleewhite shleewhite requested review from a team September 5, 2024 17:15
Copy link
Contributor

@jorytindall jorytindall left a comment

Choose a reason for hiding this comment

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

Good for me, but would also be good to get some quick eyes from @LilithJames-HDS

Copy link
Contributor

@LilithJames-HDS LilithJames-HDS left a comment

Choose a reason for hiding this comment

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

This looks great, thanks so much!

@shleewhite shleewhite merged commit eae516a into main Sep 6, 2024
16 checks passed
@shleewhite shleewhite deleted the hds-3817/badge-count-token-fix branch September 6, 2024 16:49
@hashibot-hds hashibot-hds mentioned this pull request Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants