Skip to content

[Switch] Migrate SwitchBase to emotion#24552

Merged
mnajdova merged 12 commits intomui:nextfrom
queengooborg:migrate/SwitchBase
Jan 25, 2021
Merged

[Switch] Migrate SwitchBase to emotion#24552
mnajdova merged 12 commits intomui:nextfrom
queengooborg:migrate/SwitchBase

Conversation

@queengooborg
Copy link
Contributor

@queengooborg queengooborg commented Jan 22, 2021

This PR migrates the SwitchBase component to the new emotion format as a part of #24405. Additionally, since we're here, this fixes a typo in the component's TypeScript definition for the classes (inpit -> input).

@mui-pr-bot
Copy link

mui-pr-bot commented Jan 22, 2021

Details of bundle changes

Generated by 🚫 dangerJS against 3ea2b0f

@oliviertassinari oliviertassinari added the scope: switch Changes related to the switch. label Jan 22, 2021
@oliviertassinari oliviertassinari changed the title [SwitchBase] Migrate to emotion [Switch] Migrate SwitchBase to emotion Jan 22, 2021
@queengooborg
Copy link
Contributor Author

queengooborg commented Jan 23, 2021

@oliviertassinari Am I correct to assume that private components shouldn't have an overridesResolver function, a call to useThemeProps(), or the muiName variable defined in the describeConformanceV5() of the tests?

@oliviertassinari
Copy link
Member

Am I correct to assume that private components shouldn't have an overridesResolver function, a call to useThemeProps(), or the muiName variable defined in the describeConformanceV5() of the tests?

@vinyldarkscratch Yes, this sounds about right

@oliviertassinari oliviertassinari added the on hold There is a blocker, we need to wait. label Jan 24, 2021
@oliviertassinari oliviertassinari removed the on hold There is a blocker, we need to wait. label Jan 24, 2021
@oliviertassinari
Copy link
Member

oliviertassinari commented Jan 24, 2021

I had to rebase because we needed IconButton to be migrated first.

@mnajdova mnajdova merged commit 877f62d into mui:next Jan 25, 2021
@queengooborg queengooborg deleted the migrate/SwitchBase branch January 25, 2021 07:02
natac13 pushed a commit to natac13/material-ui that referenced this pull request Jan 25, 2021
eps1lon pushed a commit to eps1lon/material-ui that referenced this pull request Jan 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: switch Changes related to the switch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants