Skip to content
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

Closed
wants to merge 13 commits into from

Conversation

Domino987
Copy link
Contributor

@Domino987 Domino987 commented Jan 5, 2022

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.

@Domino987 Domino987 changed the title Toolbar component Custom Toolbar component DateRangePicker Jan 5, 2022
@mui-pr-bot
Copy link

mui-pr-bot commented Jan 5, 2022

Details of bundle changes

Generated by 🚫 dangerJS against 0338c9e

@Domino987 Domino987 changed the title Custom Toolbar component DateRangePicker [DateRangePicker] Custom Toolbar component Jan 5, 2022
@Domino987
Copy link
Contributor Author

@siriwatknp would you mind checking this out?

Copy link
Member

@siriwatknp siriwatknp left a 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.

@flaviendelangle
Copy link
Member

I can replicate theses changes on @mui-org/x 👍

Co-authored-by: Siriwat K <siriwatkunaporn@gmail.com>
@mui-bot
Copy link

mui-bot commented Jan 13, 2022

Details of bundle changes

Generated by 🚫 dangerJS against 061fc6f

@Domino987
Copy link
Contributor Author

@flaviendelangle Should I close this then, since you copy it to the other repo?

@flaviendelangle
Copy link
Member

@Domino987 yes I think it's better that way

@Domino987 Domino987 closed this Jan 24, 2022
@Domino987 Domino987 deleted the ToolbarComponent branch January 24, 2022 11:29
@zannager zannager added the component: pickers This is the name of the generic UI component, not the React module! label Jan 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: pickers This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DateRangePicker] ToolbarComponent prop documented but not implemented
6 participants