Skip to content

[utils] Remove unnecessary excludeKeys from extractEventHandlers#48481

Merged
ZeeshanTamboli merged 7 commits into
mui:masterfrom
ZeeshanTamboli:remove-excludeKeys-from-extractEventHandlers
May 6, 2026
Merged

[utils] Remove unnecessary excludeKeys from extractEventHandlers#48481
ZeeshanTamboli merged 7 commits into
mui:masterfrom
ZeeshanTamboli:remove-excludeKeys-from-extractEventHandlers

Conversation

@ZeeshanTamboli
Copy link
Copy Markdown
Member

@ZeeshanTamboli ZeeshanTamboli commented May 6, 2026

It's not used anywhere as per my search. Not even used in MUI X: https://github.com/search?q=repo%3Amui%2Fmui-x%20extractEventHandlers&type=code.

I noticed this while reviewing a PR. extractEventHandlers is also used in useSlot method when getSlotProps is provided.

@ZeeshanTamboli ZeeshanTamboli added the package: utils Specific to the utils package. label May 6, 2026
@ZeeshanTamboli ZeeshanTamboli added the type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. label May 6, 2026
@code-infra-dashboard
Copy link
Copy Markdown

code-infra-dashboard Bot commented May 6, 2026

Deploy preview

https://deploy-preview-48481--material-ui.netlify.app/

Bundle size

Bundle Parsed size Gzip size
@mui/material ▼-64B(-0.01%) ▼-9B(-0.01%)
@mui/lab 0B(0.00%) 0B(0.00%)
@mui/private-theming 0B(0.00%) 0B(0.00%)
@mui/system 0B(0.00%) 0B(0.00%)
@mui/utils 🔺+62B(+0.41%) 🔺+35B(+0.60%)

Details of bundle changes


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

Comment thread packages/mui-utils/src/extractEventHandlers/extractEventHandlers.ts
Comment thread packages/mui-utils/src/extractEventHandlers/extractEventHandlers.ts Outdated
@ZeeshanTamboli ZeeshanTamboli requested a review from mj12albert May 6, 2026 13:19
@ZeeshanTamboli ZeeshanTamboli merged commit 67f8c0e into mui:master May 6, 2026
18 checks passed
@ZeeshanTamboli ZeeshanTamboli deleted the remove-excludeKeys-from-extractEventHandlers branch May 6, 2026 15:49
@ZeeshanTamboli ZeeshanTamboli mentioned this pull request May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: utils Specific to the utils package. 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.

3 participants