-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
fix(tabs): revert input background changes #5914
fix(tabs): revert input background changes #5914
Conversation
@tw15egan should we include this as a hotfix in a 10.11.2 release? |
@joshblack yeah would make sense to get this out ASAP |
Deploy preview for carbon-elements ready! Built with commit ff726d9 |
Deploy preview for carbon-components-react ready! Built with commit ff726d9 https://deploy-preview-5914--carbon-components-react.netlify.app |
Deploy preview for carbon-elements ready! Built with commit 04262fc |
Deploy preview for carbon-elements ready! Built with commit 5b18b78 |
Deploy preview for carbon-components-react ready! Built with commit 04262fc https://deploy-preview-5914--carbon-components-react.netlify.app |
Deploy preview for carbon-components-react ready! Built with commit 5b18b78 https://deploy-preview-5914--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.
lgtm
Co-authored-by: Josh Black <[email protected]>
Co-authored-by: Josh Black <[email protected]>
Co-authored-by: Josh Black <[email protected]>
Refs #5598
Based on a conversation in Slack, these changes were causing unintended consequences. We'll have to postpone this until v11 and also make sure we target inputs that are using the
light
prop.https://ibm-studios.slack.com/archives/C046Y0YUD/p1587564854319500
Changelog
Changed
light
prop to achieve the same effectRemoved
field-02
Testing / Reviewing
Verify nothing is broken