mui-x icon indicating copy to clipboard operation
mui-x copied to clipboard

[pickers] looser undefined value comparison

Open localyost3000 opened this issue 1 month ago • 2 comments

Explicit === null does not handle when value is undefined. using a loose equal (==) covers both null and undefined cases, preventing errors in invalid datetimes

Changelog

Properly handle luxon isValid calls when the date value is undefined (eg when a parser like fromIso fails)

localyost3000 avatar Nov 20 '25 18:11 localyost3000

Deploy preview: https://deploy-preview-20415--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 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers-pro 0B(0.00%) 0B(0.00%)
@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 :no_entry_sign: dangerJS against 3d9c75e1970df5e4ed3a00234a1222fab4b8c4e2

mui-bot avatar Nov 20 '25 18:11 mui-bot

Any :eyes: on this yet? it looks like the ci job flaked on some download, and I cannot add a label myself

localyost3000 avatar Dec 02 '25 23:12 localyost3000