Material-ui: Docs: Showcases - Layout changes when changing "Sort by"

Created on 7 Apr 2019  ·  3Comments  ·  Source: mui-org/material-ui

bug 🐛 docs good first issue

Most helpful comment

@thomasnordquist Thank you for reporting the issue, my screen was too small to reproduce it.
I'm proposing this fix:

--- a/docs/src/pages/discover-more/showcase/Showcase.js
+++ b/docs/src/pages/discover-more/showcase/Showcase.js
@@ -17,6 +17,7 @@ import appList from './appList';

 const styles = theme => ({
   root: {
+    flexGrow: 1,
     backgroundColor: theme.palette.background.default,
     // Hide the demo container padding
     margin: -theme.spacing(3),

Does it solve the problem for you? Do you want to submit a pull request? :)

All 3 comments

@thomasnordquist Thank you for reporting the issue, my screen was too small to reproduce it.
I'm proposing this fix:

--- a/docs/src/pages/discover-more/showcase/Showcase.js
+++ b/docs/src/pages/discover-more/showcase/Showcase.js
@@ -17,6 +17,7 @@ import appList from './appList';

 const styles = theme => ({
   root: {
+    flexGrow: 1,
     backgroundColor: theme.palette.background.default,
     // Hide the demo container padding
     margin: -theme.spacing(3),

Does it solve the problem for you? Do you want to submit a pull request? :)

That fix solved it for me. (I tested it in the browser)

@joshwooding thank you for testing.
I can confirm that the patch fixes the layout issue. I opened a PR (mentioned just above this comment).

Was this page helpful?
0 / 5 - 0 ratings