Skip to content

[SpeedDial] Remove deprecated props#47998

Merged
siriwatknp merged 7 commits into
mui:masterfrom
siriwatknp:remove-deprecated-SpeedDial-props
Mar 19, 2026
Merged

[SpeedDial] Remove deprecated props#47998
siriwatknp merged 7 commits into
mui:masterfrom
siriwatknp:remove-deprecated-SpeedDial-props

Conversation

@siriwatknp

Copy link
Copy Markdown
Member

Summary

Remove deprecated props from SpeedDial and SpeedDialAction.

SpeedDial

  • TransitionComponentslots.transition
  • TransitionPropsslotProps.transition

SpeedDialAction

  • FabPropsslotProps.fab
  • tooltipTitleslotProps.tooltip.title
  • tooltipPlacementslotProps.tooltip.placement
  • tooltipOpenslotProps.tooltip.open
  • TooltipClassesslotProps.tooltip.classes

Also updated SpeedDial's child cloning to read/inject placement via slotProps.tooltip.placement instead of the top-level tooltipPlacement prop.

Test plan

  • TypeScript compilation passes
  • Unit tests pass (SpeedDial: 77 tests, SpeedDialAction: 27 tests)
  • pnpm proptypes && pnpm docs:api regenerated
  • Migration guide updated

@siriwatknp siriwatknp added breaking change Introduces changes that are not backward compatible. package: material-ui Specific to Material UI. labels Mar 17, 2026
@mui-bot

mui-bot commented Mar 17, 2026

Copy link
Copy Markdown

Netlify deploy preview

Bundle size report

Bundle Parsed size Gzip size
@mui/material ▼-199B(-0.04%) ▼-81B(-0.06%)
@mui/lab 0B(0.00%) 0B(0.00%)
@mui/system 0B(0.00%) 0B(0.00%)
@mui/utils 0B(0.00%) 0B(0.00%)

Details of bundle changes

Generated by 🚫 dangerJS against 72a7173

@github-actions github-actions Bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Mar 17, 2026

@mnajdova mnajdova left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

One screenshot test still needs to be fixed.

fab: {
...fabSlotProps,
ref: createHandleSpeedDialActionButtonRef(index, origButtonRef, fabSlotOrigButtonRef),
ref: createHandleSpeedDialActionButtonRef(index, undefined, fabSlotOrigButtonRef),

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should we update createHandleSpeedDialActionButtonRef to receive only one ref now, while we are at it?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

updated

@siriwatknp siriwatknp changed the title [material-ui][SpeedDial] Remove deprecated props [SpeedDial] Remove deprecated props Mar 18, 2026
@zannager zannager added the scope: speed dial Changes related to the speed dial. label Mar 18, 2026
@github-actions github-actions Bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Mar 19, 2026

@mj12albert mj12albert left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good ~

@siriwatknp siriwatknp merged commit 8359ce0 into mui:master Mar 19, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change Introduces changes that are not backward compatible. package: material-ui Specific to Material UI. scope: speed dial Changes related to the speed dial.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants