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

[Paper] Fix color inheritance issue using nested themes #15465

Merged
merged 1 commit into from
Apr 24, 2019

Conversation

mustafahlvc
Copy link
Contributor

@mustafahlvc mustafahlvc commented Apr 23, 2019

Continue #15100

@mui-pr-bot
Copy link

Details of bundle changes.

Comparing: 387b79e...195289b

bundle parsed diff gzip diff prev parsed current parsed prev gzip current gzip
@material-ui/core +0.01% 🔺 0.00% 311,042 311,071 84,214 84,216
@material-ui/core/Paper +0.04% 🔺 +0.03% 🔺 67,279 67,308 19,970 19,976
@material-ui/core/Paper.esm +0.05% 🔺 +0.03% 🔺 60,640 60,669 18,897 18,903
@material-ui/core/Popper 0.00% 0.00% 31,115 31,115 10,793 10,793
@material-ui/core/Textarea 0.00% 0.00% 5,472 5,472 2,367 2,367
@material-ui/core/TrapFocus 0.00% 0.00% 3,731 3,731 1,565 1,565
@material-ui/core/styles/createMuiTheme 0.00% 0.00% 15,898 15,898 5,770 5,770
@material-ui/core/useMediaQuery 0.00% 0.00% 2,106 2,106 973 973
@material-ui/lab 0.00% 0.00% 140,720 140,720 42,642 42,642
@material-ui/styles 0.00% 0.00% 50,833 50,833 15,019 15,019
@material-ui/system 0.00% 0.00% 11,765 11,765 3,922 3,922
Button 0.00% 0.00% 85,622 85,622 25,618 25,618
Modal 0.00% 0.00% 79,337 79,337 23,972 23,972
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,059 648,088 202,187 202,190
packages/material-ui/build/umd/material-ui.production.min.js +0.01% 🔺 +0.01% 🔺 292,088 292,117 82,119 82,127

Generated by 🚫 dangerJS against 195289b

@oliviertassinari oliviertassinari changed the title [Paper] Fix color inheritance issue using nested themes. [Paper] Fix color inheritance issue using nested themes Apr 23, 2019
@oliviertassinari oliviertassinari added component: Paper This is the name of the generic UI component, not the React module! PR: good for merge labels Apr 23, 2019
@oliviertassinari oliviertassinari merged commit 734eca0 into mui:next Apr 24, 2019
@oliviertassinari
Copy link
Member

@mustafahlvc Well done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: Paper 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.

3 participants