Skip to content

[Tabs] Migrate Tabs to emotion#25824

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

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

Conversation

@siriwatknp

@siriwatknp siriwatknp commented Apr 18, 2021

Copy link
Copy Markdown
Member

One chunk of #24405

@mui-pr-bot

mui-pr-bot commented Apr 18, 2021

Copy link
Copy Markdown

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
Comment thread packages/material-ui/src/Tabs/Tabs.js Outdated

@oliviertassinari oliviertassinari 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.

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

Comment thread packages/material-ui/src/Tabs/Tabs.js Outdated
Comment thread packages/material-ui/src/Tabs/Tabs.js
Comment thread packages/material-ui/src/Tabs/Tabs.js
Comment thread packages/material-ui/src/Tabs/Tabs.js
Comment thread packages/material-ui/src/Tabs/Tabs.js
Comment thread packages/material-ui/src/Tabs/Tabs.js
Comment thread packages/material-ui/src/Tabs/Tabs.js Outdated
Comment thread packages/material-ui/src/Tabs/Tabs.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.

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

Comment thread packages/material-ui/src/Tabs/Tabs.js Outdated
Comment thread packages/material-ui/src/Tabs/Tabs.js Outdated
Comment thread packages/material-ui/src/Tabs/Tabs.js Outdated
Comment thread packages/material-ui/src/Tabs/Tabs.js Outdated

@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.

Few final comments

Comment thread packages/material-ui/src/Tabs/Tabs.js Outdated
Comment thread packages/material-ui/src/Tabs/Tabs.js Outdated
Comment thread packages/material-ui/src/Tabs/Tabs.js Outdated
Comment thread packages/material-ui/src/Tabs/Tabs.js Outdated
Comment thread packages/material-ui/src/Tabs/Tabs.js
Comment thread packages/material-ui/src/Tabs/Tabs.js Outdated

@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.

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
Copy Markdown
Member

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

@mnajdova

Copy link
Copy Markdown
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