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

[docs] Improve the customization demos #15368

Merged

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Apr 16, 2019

The changes are extracted from #15140. It isolates each customization example. Each customization demo is a component on its own. I'm using withStyles most of the time as it minimizes boilerplate.

@oliviertassinari oliviertassinari added the docs Improvements or additions to the documentation label Apr 16, 2019
@@ -152,4 +152,4 @@ HiddenCss.propTypes = {
xsUp: PropTypes.bool,
};

export default withStyles(styles, { name: 'MuiPrivateHiddenCss' })(HiddenCss);
export default withStyles(styles, { name: 'PrivateHiddenCss' })(HiddenCss);
Copy link
Member Author

@oliviertassinari oliviertassinari Apr 16, 2019

Choose a reason for hiding this comment

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

Move Private right to the beginning of the sheet name, it's easier to spot.

opacity: 0,
},
},
expanded: {},
Copy link
Member Author

@oliviertassinari oliviertassinari Apr 16, 2019

Choose a reason for hiding this comment

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

Follow the pseudo class convention. This state is dynamic and overrides previously defined style properties. It should have a specificity of 2.

@mui-pr-bot
Copy link

mui-pr-bot commented Apr 16, 2019

Details of bundle changes.

Comparing: 4e4a2ed...0e73597

bundle parsed diff gzip diff prev parsed current parsed prev gzip current gzip
@material-ui/core -0.00% 0.00% 315,304 315,296 85,279 85,283
@material-ui/core/Paper 0.00% 0.00% 67,277 67,277 19,975 19,975
@material-ui/core/Paper.esm 0.00% 0.00% 60,639 60,639 18,883 18,883
@material-ui/core/Popper 0.00% 0.00% 34,906 34,906 11,861 11,861
@material-ui/core/Textarea 0.00% 0.00% 5,866 5,866 2,465 2,465
@material-ui/core/styles/createMuiTheme 0.00% 0.00% 15,898 15,898 5,773 5,773
@material-ui/core/useMediaQuery 0.00% 0.00% 2,106 2,106 974 974
@material-ui/lab 0.00% 0.00% 144,182 144,182 43,530 43,530
@material-ui/styles 0.00% 0.00% 50,831 50,831 15,020 15,020
@material-ui/system 0.00% 0.00% 11,765 11,765 3,929 3,929
Button 0.00% 0.00% 88,247 88,247 26,501 26,501
Modal 0.00% 0.00% 82,524 82,524 24,821 24,821
colorManipulator 0.00% 0.00% 3,904 3,904 1,543 1,543
docs.landing 0.00% 0.00% 50,908 50,908 11,210 11,210
docs.main -0.00% -0.00% 647,004 646,998 201,944 201,943
packages/material-ui/build/umd/material-ui.production.min.js -0.00% 0.00% 291,938 291,930 82,288 82,288

Generated by 🚫 dangerJS against 0e73597

@oliviertassinari
Copy link
Member Author

I'm refactoring the ts demos to remove the usage of any.

@oliviertassinari oliviertassinari force-pushed the improve-customization-demos branch 2 times, most recently from d681097 to a8f258e Compare April 16, 2019 15:16
@oliviertassinari oliviertassinari force-pushed the improve-customization-demos branch from a8f258e to ad738d7 Compare April 16, 2019 15:25
@eps1lon eps1lon self-requested a review April 16, 2019 18:55
@eps1lon eps1lon force-pushed the improve-customization-demos branch from 48a727a to e94ac89 Compare April 16, 2019 19:22
@oliviertassinari oliviertassinari merged commit 4192017 into mui:next Apr 17, 2019
@oliviertassinari oliviertassinari deleted the improve-customization-demos branch April 17, 2019 09:41
@oliviertassinari
Copy link
Member Author

Alright, we did it! It's green, some nice isolated component demos :). @eps1lon Thanks for the help.

I hope #15140 will be the last chunk of #14999. A pull request we have broken down in many parts.

jztang pushed a commit to nyu-ossd-s19/material-ui that referenced this pull request Apr 21, 2019
* [docs] Improve the customization demos

* remove typescript usage of any

* Cast to a specific type instead of any

* fix argos
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants