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

[DatePicker][TimePicker] Add missing component declarations #29517

Merged
merged 2 commits into from
Nov 19, 2021

Conversation

longzheng
Copy link
Contributor

@longzheng longzheng commented Nov 5, 2021

The @mui/lab/themeAugmentation declarations were missing the MuiDatePicker and MuiDateTimePicker components. This made it not possible to override the props and styles for those components in a custom theme.

It appears these components were at one point part of the declarations file https://github.com/mui-org/material-ui/pull/24092/files but not sure how it was dropped or lost.

@mui-pr-bot
Copy link

mui-pr-bot commented Nov 5, 2021

No bundle size changes

Generated by 🚫 dangerJS against b3d93b2

@mbrookes mbrookes added component: date picker This is the name of the generic UI component, not the React module! typescript component: time picker This is the name of the generic UI component, not the React module! labels Nov 12, 2021
@mbrookes mbrookes changed the title Add missing component declarations for MuiDatePicker/MuiDateTimePicker [DatePicker][TimePicker] Add missing component declarations Nov 12, 2021
Copy link
Member

@michaldudak michaldudak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this addition. Would you just mind adjusting the code to keep the alphabetic order of the keys (i.e. moving the MuiDateTimePicker below MuiDateRangePickerDay)? It's good to go besides that.

@longzheng
Copy link
Contributor Author

Thanks for this addition. Would you just mind adjusting the code to keep the alphabetic order of the keys (i.e. moving the MuiDateTimePicker below MuiDateRangePickerDay)? It's good to go besides that.

Done.

@michaldudak michaldudak merged commit d145a38 into mui:master Nov 19, 2021
@longzheng longzheng deleted the patch-1 branch November 19, 2021 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: date picker This is the name of the generic UI component, not the React module! component: time picker This is the name of the generic UI component, not the React module! typescript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants