-
-
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 npm experience #38906
[docs] Improve npm experience #38906
Conversation
c13a33b
to
941d34a
Compare
Netlify deploy previewhttps://deploy-preview-38906--material-ui.netlify.app/ Bundle size report |
76983ac
to
9912a7d
Compare
9912a7d
to
c8cdfa6
Compare
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.
Looks good! Just a little copy-paste typo here. 👍
Co-authored-by: Sam Sycamore <[email protected]> Signed-off-by: Olivier Tassinari <[email protected]>
@@ -1,6 +1,6 @@ | |||
<!-- markdownlint-disable-next-line --> | |||
<p align="center"> | |||
<a href="https://mui.com/core/" rel="noopener" target="_blank"><img width="150" height="133" src="/docs/public/static/logo.svg" alt="MUI Core logo"></a> | |||
<a href="https://mui.com/core/" rel="noopener" target="_blank"><img width="150" height="133" src="https://mui.com/static/logo.svg" alt="MUI Core logo"></a> |
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.
Fix broken image in https://www.npmjs.com/package/@mui/material
# Base UI | ||
<!-- markdownlint-disable-next-line --> | ||
<p align="center"> | ||
<a href="https://mui.com/base-ui/" rel="noopener" target="_blank"><img width="150" height="133" src="https://mui.com/static/logo.svg" alt="Base UI logo"></a> |
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.
@danilo-leal We will be able to change the logos, e.g. Base UI here.
Signed-off-by: Olivier Tassinari <[email protected]> Co-authored-by: Sam Sycamore <[email protected]>
The objective is to create a nice experience like in https://www.npmjs.com/package/next. These two don't feel so great:
Preview: