Skip to content

[pickers] Merge slotProps.sectionContent in PickersInputBase#22324

Open
rin wants to merge 2 commits intomui:masterfrom
rin:fix/pickers-merge-section-content-slot-props
Open

[pickers] Merge slotProps.sectionContent in PickersInputBase#22324
rin wants to merge 2 commits intomui:masterfrom
rin:fix/pickers-merge-section-content-slot-props

Conversation

@rin
Copy link
Copy Markdown

@rin rin commented May 5, 2026

Ran into this when trying to put props on each date/time segment for accessibility (aria attributes for the spin buttons). The slotProps don't get passed through all the way, because the internal PickersInputBase is replacing the section slot props, instead of merging in any passed sectionContent slot props.

@rin rin force-pushed the fix/pickers-merge-section-content-slot-props branch from 462f711 to f7df1f4 Compare May 5, 2026 12:18
@rin rin force-pushed the fix/pickers-merge-section-content-slot-props branch from f7df1f4 to f9812aa Compare May 5, 2026 12:19
@zannager zannager added the scope: pickers Changes related to the date/time pickers. label May 5, 2026
@code-infra-dashboard
Copy link
Copy Markdown

code-infra-dashboard Bot commented May 5, 2026

Deploy preview

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

Bundle size

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 🔺+192B(+0.09%) 🔺+59B(+0.10%)
@mui/x-date-pickers-pro 🔺+185B(+0.06%) 🔺+71B(+0.09%)
@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


Check out the code infra dashboard for more information about this PR.

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants