-
Notifications
You must be signed in to change notification settings - Fork 23
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
Button v3 Tertiary Styles Update #5399
Conversation
🦋 Changeset detectedLatest commit: 260b077 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 |
#5395) * Menu: Create MenuPopover, MenuSection and MenuHeader wrappers over RAC * Update docs
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
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.
Looks hunky dory to me, potentially want to check with Jess tomorrow just to confirm from a design side tomorrow to make sure its all good in her book (generally we link out to the chromatic deploy of storybook and to the story)
* refactor: rename __future__ dir to __rc__ * refactor(Tooltip): move v1/v2 to src * refactor(Tooltip): move v3 to rc * refactor(Focusable): move into subcomponents of Tooltip * refactor(Button): move v1/v2 to src * refactor(Button): move v3 to rc * refactor(Menu): move v1/v2 to src * refactor(Menu): move v3 to rc
* chore(eslint): enable consistent-type-imports * style: fix linting errors
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.
Why
Button v3 has different styling.
https://cultureamp.atlassian.net/browse/KZN-2918
What
Updated colors for tertiary buttons
Chromatic: https://www.chromatic.com/review?appId=60a1e4a102f0cb003b5d19d6&number=5399&type=linked