Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[material-ui][Snackbar] Add Slots and SlotProps #45103

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

harry-whorlow
Copy link
Contributor

@harry-whorlow harry-whorlow commented Jan 24, 2025

part of #41281

This PR adds slot and slotProps, whilst deprecating components.

Slot:

  • slot.TransitionComponent

SlotProps:

  • slotProps.TransitionProps
  • slotProps.ContentProps,
  • slotProps.ClickAwayListenerProps

@mui-bot
Copy link

mui-bot commented Jan 24, 2025

Netlify deploy preview

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

@material-ui/core: parsed: +0.08% , gzip: +0.06%

Bundle size report

Details of bundle changes (Toolpad)
Details of bundle changes

Generated by 🚫 dangerJS against a53811c

Harry Whorlow and others added 3 commits January 24, 2025 11:11
@harry-whorlow
Copy link
Contributor Author

@DiegoAndai @siriwatknp ready for review!

/>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

One small point

This is why the docs:typescript:formatted is failing, but I'm confused as to why... As every other example of ---.tsx.preview dose not include a break line, but for some reason docs:typescript:formatted throws an error here, I'll run the command locally and it passes fine 🙃

@aarongarciah aarongarciah added component: snackbar This is the name of the generic UI component, not the React module! package: material-ui Specific to @mui/material labels Jan 24, 2025
@aarongarciah aarongarciah changed the title Add Slots and SlotProps [material-ui][Snackbar] Add Slots and SlotProps Jan 24, 2025
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: snackbar This is the name of the generic UI component, not the React module! package: material-ui Specific to @mui/material PR: out-of-date The pull request has merge conflicts and can't be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants