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] Don't suggest putting a Switch inside a ListItemSecondaryAction #23018
Conversation
Looks good, I think that we could also explain for this test case the tradeoff around |
227c614
to
d051b4a
Compare
@@ -12,7 +12,7 @@ const useStyles = makeStyles((theme: Theme) => | |||
createStyles({ | |||
root: { | |||
width: '100%', | |||
maxWidth: '36ch', | |||
maxWidth: 360, |
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.
done to have visual consistency with the other demos
@@ -9,7 +9,7 @@ const useStyles = makeStyles((theme: Theme) => | |||
root: { | |||
width: '100%', | |||
height: 400, | |||
maxWidth: 300, | |||
maxWidth: 360, |
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.
done to have visual consistency with the other demos
width={360} | ||
itemSize={46} | ||
itemCount={200} | ||
overscanCount={5} |
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.
Add overscan to limit white areas
@sirlantis It's a great first pull request on Material-UI 👌🏻. Thank you for working on it! |
@oliviertassinari Thanks, but to be honest, you did the majority of it. So: double thanks. 👍 |
Avoid future cases like #23013