Update base green 5#1246
Conversation
🦋 Changeset detectedLatest commit: f707815 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
There was a problem hiding this comment.
Pull Request Overview
This PR updates the base green 5 color value and refactors the success emphasis background color to use a design token reference instead of a hardcoded value.
- Updates base green 5 color from
#1a7f37to#1B8238 - Replaces hardcoded color value with token reference
{base.color.green.5}for bgColor-success-emphasis
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/tokens/base/color/light/light.json5 | Updates the base green 5 color value |
| src/tokens/functional/color/bgColor.json5 | Refactors success emphasis to use green 5 token reference |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
Design Token Diff (CSS)
|
Design Token Diff (StyleLint)
|
Design Token Diff (Figma)
|
|
👋 @lukasoppermann is this PR still relevant? If so, can you update the tagged reviewers? |
|
I think we won't merge it, since a11y token work is not on our plate ATM and I don't have capacity for testing |
Summary
This would fixes https://github.com/github/primer/issues/5737
What should reviewers focus on?
Steps to test: