Skip to content

[SvgIcon] Migrate to emotion#24506

Merged
oliviertassinari merged 7 commits intomui:nextfrom
oliviertassinari:svg-icon-emotion
Jan 20, 2021
Merged

[SvgIcon] Migrate to emotion#24506
oliviertassinari merged 7 commits intomui:nextfrom
oliviertassinari:svg-icon-emotion

Conversation

@oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Jan 20, 2021

One iteration on #24405. Took 31 min

@mui-pr-bot
Copy link

mui-pr-bot commented Jan 20, 2021

@material-ui/core: parsed: +0.14% , gzip: +0.13%
@material-ui/lab: parsed: +0.13% , gzip: +0.15%

Details of bundle changes

Generated by 🚫 dangerJS against 648943a

@oliviertassinari oliviertassinari added the component: SvgIcon The React component. label Jan 20, 2021
@oliviertassinari oliviertassinari marked this pull request as ready for review January 20, 2021 11:38
@oliviertassinari
Copy link
Member Author

The visual changes seem expected. We are fixing regressions we had in the past.

Co-authored-by: Marija Najdova <mnajdova@gmail.com>
}

const fullSuite = {
componentProp: testComponentProp,
Copy link
Member

Choose a reason for hiding this comment

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

👍 I think I removed this in the first iteration with the Slider, but then we decided we should keep the prop. We may need to disable this next on some of the converted components tough..

Copy link
Member Author

@oliviertassinari oliviertassinari Jan 20, 2021

Choose a reason for hiding this comment

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

It could create a conflict with open PRs, yes.

Co-authored-by: Marija Najdova <mnajdova@gmail.com>
@oliviertassinari oliviertassinari merged commit 1b9eed2 into mui:next Jan 20, 2021
@oliviertassinari oliviertassinari deleted the svg-icon-emotion branch January 20, 2021 13:09
@queengooborg queengooborg mentioned this pull request Jan 20, 2021
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: SvgIcon The React component.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants