-
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
fix(data-table): remove inline edit short table styles #8212
fix(data-table): remove inline edit short table styles #8212
Conversation
Deploy preview for carbon-elements ready! Built with commit a520fc8 |
Deploy preview for carbon-components-react ready! Built without sensitive environment variables with commit a520fc8 https://deploy-preview-8212--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 👍 ✅
9e6b0b6
to
901c0f2
Compare
DCO Assistant Lite bot: Thanks for your submission! We ask that you sign our Developer Certificate of Origin before we can accept your contribution. You can sign the DCO by adding a comment below using this text: I have read the DCO document and I hereby sign the DCO. You can retrigger this bot by commenting recheck in this Pull Request |
901c0f2
to
735fa03
Compare
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!
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 goood!
Closes #8206
This PR removes broken style overrides for form controls within short data tables
Testing / Reviewing
Confirm that inline selects within the short data table have the correct spacing and alignment for the chevron icon. The extra table column should be removed from the examples before merging