-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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
[DateRangePicker] Custom Toolbar component #30514
Conversation
@siriwatknp would you mind checking this out? |
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.
It looks good to me. I will leave @flaviendelangle to decide because he is mainly working on the DatePickers.
I can replicate theses changes on @mui-org/x 👍 |
Co-authored-by: Siriwat K <siriwatkunaporn@gmail.com>
@flaviendelangle Should I close this then, since you copy it to the other repo? |
@Domino987 yes I think it's better that way |
Fixes mui/mui-x#4557
The ToolbarComponent prop for overriding the toolbar is documented for the DateRangePicker, but is ignored in the code.
This PRs updates the types to correctly display the DateRangePicker Toolbar props and renders the custom toolbar if provided.
I also added some base tests for the date range picker as well as a test for the Toolbar override.