-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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] Add small formatting improvements to the licensing page #11178
Conversation
Deploy preview: https://deploy-preview-11178--material-ui-x.netlify.app/ Updated pages: |
|
||
The Pro version is available under a commercial license—visit [the Pricing page](https://mui.com/r/x-get-license/) for details. | ||
Exclusive features are marked with the <span class="plan-pro" aria-label="MUI X Pro plan icon"></span> icon throughout the documentation. |
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.
beyond the scope of this PR but we should track down all of these lil plan spans and give them all aria labels where they're missing
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.
I checked out these spans in the navigation bar and they all appear to have a title
that the screen reader picks up as an ARIA label, but I suspect there may be others scattered around in the Markdown that don't have them.
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.
For sure, yeah! I'm just now getting deeper into the X docs space, so definitely looking forward to doing it!
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 found one more awkward sentence in this pass :)
While working on mui/material-ui#39832, I noticed a few relatively quick improvement opportunities for the MUI X docs! This one adds in just some minor copywriting and formatting tweaks to make the page more consistent:
👉 https://deploy-preview-11178--material-ui-x.netlify.app/x/introduction/licensing/