-
Notifications
You must be signed in to change notification settings - Fork 78
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
feat(table): enable single nested hierarchy styling #1609
Conversation
Deploy preview for carbon-addons-iot-react ready! Built with commit 1b22d2b https://deploy-preview-1609--carbon-addons-iot-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.
Nice work! I give an approval right away, but please consider the comment regarding the new prop and its naming
…s-iot-react into table-hierarchy
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 good to me
…s-iot-react into table-hierarchy
🎉 This PR is included in version 2.122.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Closes #1602
Closes #1600
Summary
hasSingleNestedHierarchy
as an option that will disable the gray offset if enabledRelated internal Monitor design: https://ibm.invisionapp.com/share/KWO0FZ5BHQ9#/screens/319695909
Change List (commits, features, bugs, etc)
hasSingleNestedHierarchy
propAcceptance Test (how to verify the PR)
Regression Test (how to make sure I didn't break things)
Go to ?path=/story/watson-iot-table--stateful-example-with-row-nesting-and-fixed-columns
Go to table resize AND fixed column stories to ensure that the padding still looks correct in the select all checkbox in the table head AND the offset still works correctly with those (if they include nested rows)