Skip to content

[internal][pickers] Don't validate inputFormat in production#26053

Merged
eps1lon merged 1 commit intomui:nextfrom
eps1lon:feat/pickers/disableCloseOnSelect
Apr 29, 2021
Merged

[internal][pickers] Don't validate inputFormat in production#26053
eps1lon merged 1 commit intomui:nextfrom
eps1lon:feat/pickers/disableCloseOnSelect

Conversation

@eps1lon
Copy link
Copy Markdown
Member

@eps1lon eps1lon commented Apr 29, 2021

Every implementation provides a default value for inputFormat. The current invariant would therefore only validate our own implementation and we generally don't do this. Especially since usePickerState does not use the inputFormat.

@eps1lon eps1lon added performance scope: pickers Changes related to the date/time pickers. labels Apr 29, 2021
@mui-pr-bot
Copy link
Copy Markdown

mui-pr-bot commented Apr 29, 2021

Details of bundle changes

Generated by 🚫 dangerJS against 99e2fe1

@eps1lon eps1lon marked this pull request as ready for review April 29, 2021 13:50
ampmInClock: true,
orientation,
showToolbar: true,
showTabs: true,
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This prop was unused.

@eps1lon eps1lon merged commit 824dca2 into mui:next Apr 29, 2021
@eps1lon eps1lon deleted the feat/pickers/disableCloseOnSelect branch April 29, 2021 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance scope: pickers Changes related to the date/time pickers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants