-
-
Notifications
You must be signed in to change notification settings - Fork 32.7k
Closed
Labels
breaking changeIntroduces changes that are not backward compatible.Introduces changes that are not backward compatible.ready to takeHelp wanted. Guidance available. There is a high chance the change will be acceptedHelp wanted. Guidance available. There is a high chance the change will be acceptedscope: accordionChanges related to the accordion.Changes related to the accordion.
Description
- I have searched the issues of this repository and believe that this is not a duplicate.
Summary 💡
For component packages/material-ui/src/AccordionDetails/AccordionDetails.js. https://github.com/mui/material-ui/blob/next/packages/material-ui/src/AccordionDetails/AccordionDetails.js#L9
I don't see any reason of setting a display: flex onto the root div. The display: block style (on top of the fact that it does not require explicit setting) might even be preferred.
Unfortunately this is a breaking change :/.
Examples 🌈
Motivation 🔦
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
breaking changeIntroduces changes that are not backward compatible.Introduces changes that are not backward compatible.ready to takeHelp wanted. Guidance available. There is a high chance the change will be acceptedHelp wanted. Guidance available. There is a high chance the change will be acceptedscope: accordionChanges related to the accordion.Changes related to the accordion.