Skip to content

[Accordion] Remove display:flex from AccordionDetails #22456

@croraf

Description

@croraf
  • 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 🔦

Metadata

Metadata

Assignees

No one assigned

    Labels

    breaking changeIntroduces changes that are not backward compatible.ready to takeHelp wanted. Guidance available. There is a high chance the change will be acceptedscope: accordionChanges related to the accordion.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions