-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
style(button): fix various button bugs #7029
Conversation
7a794c4
to
ec74c9f
Compare
Deploy preview for carbon-elements ready! Built with commit 7a794c4 |
Deploy preview for carbon-components-react ready! Built with commit 7a794c4 https://deploy-preview-7029--carbon-components-react.netlify.app |
Deploy preview for carbon-elements ready! Built with commit 479d473 |
Deploy preview for carbon-components-react ready! Built without sensitive environment variables with commit 479d473 https://deploy-preview-7029--carbon-components-react.netlify.app |
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.
Everything looks great in all themes and tested in safari/chrome/firefox! Thank you so much for fixing all of these so fast 🚀
5bacdf4
to
832ee56
Compare
Is this ready to be merged @tw15egan ? |
bump @joshblack |
Closes #7021
Closes #7025
Closes #7026
Various bug fixes for style issues found in
Button
.Changelog
New
Danger
variant to Playground:active
styles forTertiary Button
([button] tertiary button has active state bug in Chrome browser #7026)disabled
class is present ([button] disabled states have bugs #7021)Changed
ButtonSet
styles to after the@include button-base;
mixin call that is applied tobx--btn
. This was causing styling issues as things were not properly cascading down. ([button] disabled states have bugs #7021)$disabled
tokens with$disabled-03
([button] disabled states have bugs #7021)$ui-02
to$ui-background
([button] inset focus needs to be transparent #7025)Removed
disabled
style block that was causing wrong values to be usedTesting / Reviewing
disabled
knob and ensuredisabled
styles are correct across all variants