-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[DataGrid] Rename hideToolbar prop to showToolbar #706
[DataGrid] Rename hideToolbar prop to showToolbar #706
Conversation
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.
For the documentation, I think that we could document displaying the de density toolbar before the density prop. How about this structure?
Density selector
How do enable the toolbar,
Demo
Have a note about how to hide the selector when the toolbar is visible, no demo for it
Density prop
Demo
docs/src/pages/components/data-grid/getting-started/getting-started.md
Outdated
Show resolved
Hide resolved
@mbrookes can I merge this? |
Following up to the Density selector PR #606 I'm renaming the
hideToolbar
prop toshowToolbar
to make the toolbar opt-in.This decision was based on both discussions that happened on the #606 PR and in Slack.