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
update(react-card): dark mode colors #992
Conversation
π¦ Changeset detectedLatest commit: b4b36c9 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 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 |
The latest updates on your projects. Learn more about Vercel for Git βοΈ
|
π¦ Canary Packages PublishedLatest commit: b4b36c9 Published 2 packages@hashicorp/react-card@0.13.0-canary-20230811213229
@hashicorp/react-related-content@0.3.6-canary-20230811213229
|
Moving away from a 'expected token as variable, value as fallback' approach, in favor of a 'value used directly, design token in comment' approach. Ref: https://hashicorp.slack.com/archives/C031LA4KS4S/p1691774993436509?thread_ts=1691762754.792629&cid=C031LA4KS4S
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.
LGTM provided we adjust the changelog!
Co-authored-by: Dylan Staley <88163+dstaley@users.noreply.github.com>
ποΈ Discussion
π Preview Link
Description
(EDIT By @EnMod:) This PR updates the colors of dark appearance
<Card />
components to match our new spec, using new tokens. Prompted by a Vercel thread on using new colors for cards on an upcoming project (discussion thread linked above). Design has approved of the look/feel.PR Checklist π
Items in this checklist may not may not apply to your PR, but please consider each item carefully.