Skip to content

[Accordion] Migrate AccordionDetails to emotion#24539

Merged
mnajdova merged 2 commits into
mui:nextfrom
queengooborg:migrate/AccordionDetails
Jan 22, 2021
Merged

[Accordion] Migrate AccordionDetails to emotion#24539
mnajdova merged 2 commits into
mui:nextfrom
queengooborg:migrate/AccordionDetails

Conversation

@queengooborg

Copy link
Copy Markdown
Contributor

This PR migrates the AccordionDetails component to the new emotion format as a part of #24405.

@mui-pr-bot

mui-pr-bot commented Jan 21, 2021

Copy link
Copy Markdown

@material-ui/core: parsed: +0.09% , gzip: +0.04%

Details of bundle changes

Generated by 🚫 dangerJS against ee96fe6

@queengooborg

Copy link
Copy Markdown
Contributor Author

Looks like another potential unrelated test failure... :(

@oliviertassinari oliviertassinari changed the title [AccordionDetails] Migrate to emotion [Accordion] Migrate AccordionDetails to emotion Jan 21, 2021
@oliviertassinari oliviertassinari added the scope: accordion Changes related to the accordion. label Jan 21, 2021
@mnajdova

Copy link
Copy Markdown
Member

Looks like another potential unrelated test failure... :(

I did a re-run

Comment thread packages/material-ui/src/AccordionDetails/AccordionDetails.js

@mnajdova mnajdova left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just one comment, looks good otherwise :) 👍

@mnajdova
mnajdova merged commit 8b2a5f7 into mui:next Jan 22, 2021
@queengooborg
queengooborg deleted the migrate/AccordionDetails branch January 22, 2021 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: accordion Changes related to the accordion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants