-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Modified Scraps Graphics Token Values #105598
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
Conversation
| ...tokens.graphics, | ||
| muted: tokens.graphics.neutral.moderate, | ||
| // Apply compatability Proxy to deprecated tokens | ||
| muted: createBackwardsCompatibleToken(tokens.graphics.neutral), |
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.
Graphics muted token now returns wrong color value
The change from tokens.graphics.neutral.moderate to createBackwardsCompatibleToken(tokens.graphics.neutral) alters the legacy behavior of graphics.muted. Previously, when used as a string, it returned the moderate color. Now the proxy returns vibrant when coerced to string (per createBackwardsCompatibleToken implementation). The deprecation comment also states graphics.muted should map to graphics.neutral.moderate, not vibrant. This will cause existing usages of graphics.muted to render a different color.
|
Closing in favor of #105641 |
Replaces #105598. Updates all variants of the `tokens.graphics` and `tokens.border.muted`. See the breadcrumb text and icons examples below. | Before | After | |--------|--------| | <img width="1280" alt="tokens-light-old" src="https://github.com/user-attachments/assets/1cb4b024-5991-4f8e-90a5-407ee59756ff" /> | <img width="1280" alt="tokens-light-new" src="https://github.com/user-attachments/assets/6a3ca496-e121-4da3-9e93-8c76f7e13a16" /> | | <img width="1280" alt="tokens-dark-old-1" src="https://github.com/user-attachments/assets/a8a679f1-cef7-4103-bb04-220bee0e5da2" /> | <img width="1280" alt="tokens-dark-new" src="https://github.com/user-attachments/assets/f7e9289f-0c4d-47f5-bdf2-a2431fa05297" />
Replaces #105598. Updates all variants of the `tokens.graphics` and `tokens.border.muted`. See the breadcrumb text and icons examples below. | Before | After | |--------|--------| | <img width="1280" alt="tokens-light-old" src="https://github.com/user-attachments/assets/1cb4b024-5991-4f8e-90a5-407ee59756ff" /> | <img width="1280" alt="tokens-light-new" src="https://github.com/user-attachments/assets/6a3ca496-e121-4da3-9e93-8c76f7e13a16" /> | | <img width="1280" alt="tokens-dark-old-1" src="https://github.com/user-attachments/assets/a8a679f1-cef7-4103-bb04-220bee0e5da2" /> | <img width="1280" alt="tokens-dark-new" src="https://github.com/user-attachments/assets/f7e9289f-0c4d-47f5-bdf2-a2431fa05297" />
vibrantgraphics groupcontentto includecreateBackwardsComaptibleTokenformutedScreenshots
Before Light
Before Dark
After Light
After Dark