-
-
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] Accessibility in Base UI #39264
Conversation
8b2fe22
to
bf07148
Compare
Netlify deploy previewBundle size report |
1b3eaf4
to
e44d9c0
Compare
Co-authored-by: Sam Sycamore <[email protected]> Signed-off-by: Michał Dudak <[email protected]>
@samuelsycamore, thanks for your corrections! I included all of 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.
Love this PR! This page is super important 😊 Left a few considerations about adding demos to illustrate the concepts we described — avoiding the page feeling like a wall of text.
Co-authored-by: Danilo Leal <[email protected]> Signed-off-by: Michał Dudak <[email protected]>
@danilo-leal, I added the demos. They may need to be updated after #38820 is merged in. |
Love the demos! They do a great job of driving home the key points. |
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.
Woo, this is great! Lovely addition to the Base UI docs 🤙
Signed-off-by: Michał Dudak <[email protected]> Co-authored-by: Sam Sycamore <[email protected]> Co-authored-by: Danilo Leal <[email protected]>
Described how Base UI components are accessible and what the developers need to do on their side when building applications.
@samuelsycamore, feel free to edit or expand where you feel necessary.
Closes #38539
👉 https://deploy-preview-39264--material-ui.netlify.app/base-ui/getting-started/accessibility/