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] Fix generated proptypes #45156

Merged
merged 3 commits into from
Feb 4, 2025

Conversation

siriwatknp
Copy link
Member

#45103 produced noise in the proptypes slotProps.clickAwayListener due to the structure of the props does not contain html attributes so the script tries to show all possible values.

I fix it by using the typeof ClickAwayListener instead to remove the ReactElement from generating as proptypes.

@siriwatknp siriwatknp 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 31, 2025
@siriwatknp siriwatknp requested a review from sai6855 January 31, 2025 08:14
@mui-bot
Copy link

mui-bot commented Jan 31, 2025

Netlify deploy preview

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

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 590a726

@sai6855
Copy link
Contributor

sai6855 commented Jan 31, 2025

@siriwatknp can you run pnpm docs:api, looks like CI is failing because of that

@oliviertassinari oliviertassinari added bug 🐛 Something doesn't work regression A bug, but worse labels Feb 2, 2025
@siriwatknp
Copy link
Member Author

@siriwatknp can you run pnpm docs:api, looks like CI is failing because of that

Done, thanks.

@siriwatknp siriwatknp requested a review from a team February 3, 2025 13:18
Copy link
Contributor

@sai6855 sai6855 left a comment

Choose a reason for hiding this comment

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

👍

@siriwatknp siriwatknp merged commit 73b3169 into mui:master Feb 4, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work component: snackbar This is the name of the generic UI component, not the React module! package: material-ui Specific to @mui/material regression A bug, but worse
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants