Skip to content

Conversation

@siriwatknp
Copy link
Member

@siriwatknp siriwatknp commented Oct 31, 2025

Issue

Reproduction: https://stackblitz.com/edit/1kmt3frx?file=src%2FDemo.tsx

While checking #20040, I found another bug shown in the recording below.
The value return from onChange callback is swapping between null and invalid date.

Screen.Recording.2568-10-31.at.11.58.49.mov

Fix

It should check for previous invalid date, not valid.

Test added.

@siriwatknp siriwatknp added type: bug It doesn't behave as expected. scope: pickers Changes related to the date/time pickers. labels Oct 31, 2025
@mui-bot
Copy link

mui-bot commented Oct 31, 2025

Deploy preview: https://deploy-preview-20158--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-charts-premium 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers ▼-25B(-0.01%) ▼-12B(-0.02%)
@mui/x-date-pickers-pro ▼-27B(-0.01%) ▼-12B(-0.01%)
@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 d14f117

@siriwatknp siriwatknp requested a review from a team October 31, 2025 05:06
@siriwatknp siriwatknp marked this pull request as draft October 31, 2025 05:16
@siriwatknp siriwatknp removed the request for review from a team October 31, 2025 05:16
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: bug It doesn't behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants