Skip to content

Conversation

LukasTy
Copy link
Member

@LukasTy LukasTy commented Jul 30, 2025

Fix #14684.
Fix #14284.

Apply this proposal: #14684 (comment)

@LukasTy LukasTy self-assigned this Jul 30, 2025
@LukasTy LukasTy added scope: pickers Changes related to the date/time pickers. type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. labels Jul 30, 2025
@mui-bot
Copy link

mui-bot commented Jul 30, 2025

Deploy preview: https://deploy-preview-18980--material-ui-x.netlify.app/

Bundle size report

Bundle Parsed size Gzip size
@mui/x-data-grid 0B(0.00%) 0B(0.00%)
@mui/x-data-grid-pro 0B(0.00%) 0B(0.00%)
@mui/x-data-grid-premium 0B(0.00%) 0B(0.00%)
@mui/x-charts 0B(0.00%) 0B(0.00%)
@mui/x-charts-pro 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers 🔺+665B(+0.29%) 🔺+229B(+0.37%)
@mui/x-date-pickers-pro 🔺+666B(+0.21%) 🔺+216B(+0.26%)
@mui/x-tree-view 0B(0.00%) 0B(0.00%)
@mui/x-tree-view-pro 0B(0.00%) 0B(0.00%)

Details of bundle changes

Generated by 🚫 dangerJS against fbc8059

@LukasTy LukasTy changed the title [pickers] Handle textField.slotProps gracefully [pickers] Gracefully handle textField.slotProps Jul 31, 2025
@LukasTy LukasTy requested a review from flaviendelangle July 31, 2025 11:32
? {
slotProps: {
...other?.slotProps,
input: (ownerState: FieldOwnerState) => ({
Copy link
Member

Choose a reason for hiding this comment

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

Would it work with the resolveComponentProps utility function instead?

Copy link
Member Author

Choose a reason for hiding this comment

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

I've updated the solution to use resolveComponentProps .
Also made some other fixes testing various cases and added some tests to ensure at least a semi-stability of this tricky and fragile behavior (until we drop support for Material v5).


Besides, it looks like this now solves #14284 as well. 🎉

Copy link
Member

@flaviendelangle flaviendelangle left a comment

Choose a reason for hiding this comment

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

Big cleanup here! 💙

@LukasTy LukasTy merged commit 40d9b07 into mui:master Sep 10, 2025
21 checks passed
@LukasTy LukasTy deleted the fix-14684 branch September 10, 2025 07:49
JCQuintas pushed a commit to JCQuintas/mui-x that referenced this pull request Sep 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: pickers Changes related to the date/time pickers. type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[pickers] DatePicker Error - Cannot read properties of null (reading 'selectionStart') [pickers] Handle TextField slots with @mui/material@6

3 participants