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

[Radio][Checkbox] Revert breaking changes #15483

Merged
merged 3 commits into from
Apr 26, 2019

Conversation

oliviertassinari
Copy link
Member

I'm reverting part of the changes done in #15097.

Closes #15470

@oliviertassinari oliviertassinari added bug 🐛 Something doesn't work component: checkbox This is the name of the generic UI component, not the React module! component: radio This is the name of the generic UI component, not the React module! labels Apr 25, 2019
@oliviertassinari oliviertassinari changed the title [Radio][Checkbox] Revert breaking change [Radio][Checkbox] Revert breaking changes Apr 25, 2019
@oliviertassinari oliviertassinari changed the title [Radio][Checkbox] Revert breaking changes [WIP][Radio][Checkbox] Revert breaking changes Apr 25, 2019
@mui-pr-bot
Copy link

mui-pr-bot commented Apr 25, 2019

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

Details of bundle changes.

Comparing: 1155be4...f5ac6bf

bundle parsed diff gzip diff prev parsed current parsed prev gzip current gzip
@material-ui/core +0.29% 🔺 +0.06% 🔺 311,191 312,087 84,408 84,457
@material-ui/core/Paper 0.00% 0.00% 67,308 67,308 19,985 19,985
@material-ui/core/Paper.esm 0.00% 0.00% 60,714 60,714 18,898 18,898
@material-ui/core/Popper 0.00% 0.00% 31,113 31,113 10,801 10,801
@material-ui/core/Textarea 0.00% 0.00% 5,472 5,472 2,366 2,366
@material-ui/core/TrapFocus 0.00% 0.00% 3,731 3,731 1,564 1,564
@material-ui/core/styles/createMuiTheme 0.00% 0.00% 15,943 15,943 5,778 5,778
@material-ui/core/useMediaQuery 0.00% 0.00% 2,106 2,106 974 974
@material-ui/lab 0.00% 0.00% 140,760 140,760 42,560 42,560
@material-ui/styles 0.00% 0.00% 50,835 50,835 15,023 15,023
@material-ui/system 0.00% 0.00% 11,765 11,765 3,923 3,923
Button 0.00% 0.00% 85,663 85,663 25,640 25,640
Modal 0.00% 0.00% 20,578 20,578 6,609 6,609
colorManipulator 0.00% 0.00% 3,904 3,904 1,543 1,543
docs.landing 0.00% 0.00% 51,154 51,154 11,250 11,250
docs.main 0.00% 0.00% 648,872 648,872 202,332 202,332
packages/material-ui/build/umd/material-ui.production.min.js +0.29% 🔺 +0.08% 🔺 292,685 293,520 82,275 82,339

Generated by 🚫 dangerJS against f5ac6bf

@oliviertassinari oliviertassinari changed the title [WIP][Radio][Checkbox] Revert breaking changes [Radio][Checkbox] Revert breaking changes Apr 25, 2019
- [Switch][Radio][Checkbox] Improve specification compliance.

Refactore the implementation to make it easier to override the styles.
- [Switch] Refactore the implementation to make it easier to override the styles.
Copy link
Member Author

Choose a reason for hiding this comment

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

I have reverted the specificity increase. You can see the impact on the customization examples. I believe the Radio and Checkbox changes are no longer breaking.

@oliviertassinari
Copy link
Member Author

@joshwooding Thank you for the review.

@oliviertassinari oliviertassinari merged commit 72f4f9f into mui:next Apr 26, 2019
@oliviertassinari oliviertassinari deleted the radio-checkbox-revert branch April 26, 2019 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work component: checkbox This is the name of the generic UI component, not the React module! component: radio This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Radio][Checkbox] wrong color when disabled
3 participants