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] Introduce the slots API #34873

Merged
merged 17 commits into from Nov 3, 2022

Conversation

michaldudak
Copy link
Member

@michaldudak michaldudak commented Oct 24, 2022

As discussed in #33416.
This PR introduces the slots and slotProps props in the components that have components and componentsProps, respectively. The new props work alongside the old ones. Where both old and new API is used, the new one takes precedence.

This change is not supposed to break any existing codebases. It only adds new props without modifying the behavior of the old ones.

The PR also updates the conformance tests to make sure the new API works consistently across all components.

@michaldudak michaldudak added package: material-ui Specific to @mui/material enhancement This is not a bug, nor a new feature labels Oct 24, 2022
@mui-bot
Copy link

mui-bot commented Oct 24, 2022

Messages
📖 Netlify deploy preview: https://deploy-preview-34873--material-ui.netlify.app/

Details of bundle changes

@material-ui/core: parsed: +0.27% , gzip: +0.47%
@material-ui/lab: parsed: +0.27% , gzip: +0.39%

Generated by 🚫 dangerJS against 951ee26

Copy link
Member

@mnajdova mnajdova left a comment

Choose a reason for hiding this comment

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

I can't notice anything out of place.

@michaldudak michaldudak merged commit 547b760 into mui:master Nov 3, 2022
@michaldudak michaldudak deleted the slots-in-material-ui branch November 3, 2022 08:44
daniel-rabe pushed a commit to daniel-rabe/material-ui that referenced this pull request Nov 29, 2022
feliperli pushed a commit to jesrodri/material-ui that referenced this pull request Dec 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This is not a bug, nor a new feature package: material-ui Specific to @mui/material
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants