-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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
[docs] Improve the customization demos #15368
Conversation
@@ -152,4 +152,4 @@ HiddenCss.propTypes = { | |||
xsUp: PropTypes.bool, | |||
}; | |||
|
|||
export default withStyles(styles, { name: 'MuiPrivateHiddenCss' })(HiddenCss); | |||
export default withStyles(styles, { name: 'PrivateHiddenCss' })(HiddenCss); |
There was a problem hiding this comment.
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: {}, |
There was a problem hiding this comment.
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.
Details of bundle changes.Comparing: 4e4a2ed...0e73597
|
I'm refactoring the ts demos to remove the usage of any. |
d681097
to
a8f258e
Compare
a8f258e
to
ad738d7
Compare
48a727a
to
e94ac89
Compare
* [docs] Improve the customization demos * remove typescript usage of any * Cast to a specific type instead of any * fix argos
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.