Skip to content

[Tabs] Migrate Tabs to emotion#25824

Merged
mnajdova merged 15 commits intomui:nextfrom
siriwatknp:tabs-migrate-emotion
Apr 23, 2021
Merged

[Tabs] Migrate Tabs to emotion#25824
mnajdova merged 15 commits intomui:nextfrom
siriwatknp:tabs-migrate-emotion

Conversation

@siriwatknp
Copy link
Member

@siriwatknp siriwatknp commented Apr 18, 2021

One chunk of #24405

@mui-pr-bot
Copy link

mui-pr-bot commented Apr 18, 2021

Details of bundle changes

@material-ui/core: parsed: +0.34% , gzip: +0.22%
@material-ui/lab: parsed: +0.41% , gzip: +0.28%

Generated by 🚫 dangerJS against 1cc92d9

@siriwatknp siriwatknp changed the title [Tabs] migrate Tabs to emotion [Tabs] Migrate Tabs to emotion Apr 18, 2021
@siriwatknp

This comment has been minimized.

@mnajdova mnajdova mentioned this pull request Apr 18, 2021
1 task
@oliviertassinari oliviertassinari added the scope: tabs Changes related to the tabs. label Apr 18, 2021
Copy link
Member

@oliviertassinari oliviertassinari left a comment

Choose a reason for hiding this comment

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

At a high level, it looks good, but I didn't dive deep into the review. Best to have Marija review it

Copy link
Member

@mnajdova mnajdova left a comment

Choose a reason for hiding this comment

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

Good job, couple of things to be resolved before merging.

siriwatknp and others added 3 commits April 22, 2021 08:57
Co-authored-by: Marija Najdova <mnajdova@gmail.com>
Co-authored-by: Marija Najdova <mnajdova@gmail.com>
Copy link
Member

@mnajdova mnajdova left a comment

Choose a reason for hiding this comment

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

Few final comments

siriwatknp and others added 4 commits April 23, 2021 00:09
Co-authored-by: Marija Najdova <mnajdova@gmail.com>
Co-authored-by: Marija Najdova <mnajdova@gmail.com>
Co-authored-by: Marija Najdova <mnajdova@gmail.com>
Copy link
Member

@mnajdova mnajdova left a comment

Choose a reason for hiding this comment

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

I just pushed one fix for the scrollButtons overrides, looks good otherwise 👍

@mnajdova mnajdova merged commit 7f3653e into mui:next Apr 23, 2021
@oliviertassinari
Copy link
Member

@material-ui/core => 100% migrated 🙏

@mnajdova
Copy link
Member

@material-ui/core => 100% migrated 🙏

Looks like we both waited to see this too long now 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: tabs Changes related to the tabs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants