Skip to content
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][material-ui] Adjust slightly the installation page content #38380

Merged
merged 1 commit into from
Aug 8, 2023

Conversation

danilo-leal
Copy link
Contributor

Some small tweaks I thought of doing while quickly browsing the Material UI installation page:

  • The callout about the use client directive seems more like a recommendation than a warning. Also, there's something psychological about going to the Installation page, and the very first thing you see is a "warning" message... not great, I think. Makes me feel cautious about going in to install this thing.
  • Removed the warning callout from a piece of content that could just not be in a callout entirely (recommends going to another page for more details).
  • Standardized the font installation copy to be consistent with the Joy UI one.

@danilo-leal danilo-leal added docs Improvements or additions to the documentation package: material-ui Specific to @mui/material labels Aug 8, 2023
@danilo-leal danilo-leal self-assigned this Aug 8, 2023
Copy link
Member

@samuelsycamore samuelsycamore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call on all counts!

@danilo-leal danilo-leal merged commit 7961563 into master Aug 8, 2023
26 checks passed
@danilo-leal danilo-leal deleted the material-ui-installation-tweaks branch August 8, 2023 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation package: material-ui Specific to @mui/material
Projects
Status: Recently completed
Development

Successfully merging this pull request may close these issues.

None yet

2 participants