-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
[docs] Remove todo link from sidebar #37373
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
Conversation
Netlify deploy previewhttps://deploy-preview-37373--material-ui.netlify.app/ Bundle size report |
a647384
to
c315e83
Compare
c315e83
to
6fe1429
Compare
I think that it would be great to copy what was great about the older page into the new ones. I believe that are 2 things: #35213 (comment). |
That was the idea but to move the docs for |
6fe1429
to
f4f61b9
Compare
f4f61b9
to
0455225
Compare
I am ok with including the changes linked in Olivier's PR in the next PR. |
Note, we will still need the pages in the Material UI docs, for people not familiar too much with how the @mui/system is related it could be great to have the content in the docs where they browser most of the features. |
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.
Thanks!
PR for
CssVarsProvider
documentation separately