Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Pagination] Migrate Pagination and PaginationItem to emotion #25146

Merged
merged 22 commits into from
Mar 1, 2021

Conversation

mngu
Copy link
Contributor

@mngu mngu commented Feb 28, 2021

One chunk of #24405

@mui-pr-bot
Copy link

mui-pr-bot commented Feb 28, 2021

@material-ui/core: parsed: +0.29% , gzip: +0.22%
@material-ui/lab: parsed: +0.29% , gzip: +0.22%

Details of bundle changes

Generated by 🚫 dangerJS against fdacf2d

@oliviertassinari oliviertassinari changed the title [Pagination][PaginationItem] Migrate to emotion [Pagination] Migrate PaginationItem to emotion Feb 28, 2021
@oliviertassinari oliviertassinari added component: pagination This is the name of the generic UI component, not the React module! new feature New feature or request labels Feb 28, 2021
@oliviertassinari oliviertassinari changed the title [Pagination] Migrate PaginationItem to emotion [Pagination] Migrate Pagination and PaginationItem to emotion Feb 28, 2021
@mngu
Copy link
Contributor Author

mngu commented Mar 1, 2021

@oliviertassinari thx for correcting this !

@oliviertassinari
Copy link
Member

@mngu no problem, it's meant as a review. It's faster to commit directly with a message that explain why so that you can review the changes.

@oliviertassinari oliviertassinari merged commit ab16c85 into mui:next Mar 1, 2021
@oliviertassinari
Copy link
Member

@mngu Thanks for spending time on this, the last few pull requests have been helpful :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: pagination This is the name of the generic UI component, not the React module! new feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants