Skip to content

[Backdrop] Migrate to emotion#24523

Merged
mnajdova merged 5 commits into
mui:nextfrom
queengooborg:migrate/Backdrop
Jan 21, 2021
Merged

[Backdrop] Migrate to emotion#24523
mnajdova merged 5 commits into
mui:nextfrom
queengooborg:migrate/Backdrop

Conversation

@queengooborg

Copy link
Copy Markdown
Contributor

This PR migrates the Backdrop 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.11% , gzip: +0.06%
@material-ui/lab: parsed: +0.09% , gzip: +0.08%

Details of bundle changes

Generated by 🚫 dangerJS against 94c36d4

@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 a couple of comments otherwise looks good 👍

Comment thread packages/material-ui/src/Backdrop/Backdrop.js Outdated
Comment thread packages/material-ui/src/Backdrop/Backdrop.test.js Outdated
@mnajdova mnajdova added the scope: backdrop This is the name of the generic UI component, not the React module! label Jan 21, 2021
@queengooborg

Copy link
Copy Markdown
Contributor Author

Looks like the package build from CodeSandbox failed, seems like another fluke. Needs a quick rerun, and all should be good!

@github-actions github-actions Bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Jan 21, 2021
@github-actions github-actions Bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Jan 21, 2021
Comment thread docs/pages/api-docs/backdrop.json Outdated
@mnajdova
mnajdova merged commit 302c303 into mui:next Jan 21, 2021
@queengooborg
queengooborg deleted the migrate/Backdrop branch January 21, 2021 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: backdrop This is the name of the generic UI component, not the React module!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants