-
-
Notifications
You must be signed in to change notification settings - Fork 9.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
[addon-a11y] Fix best practices added in #14736 #14754
Conversation
a613300
to
eed779a
Compare
Nx Cloud ReportCI ran the following commands for commit 71cfb5b. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this branch
Sent with 💌 from NxCloud. |
@tooppaaa @adagar @jsomsanith any chance you can give a thorough review here? will be great to get more a11y process stuff documented for storybook 🙏 |
Thanks for this awesome documentation and guidelines @kaelig ! |
Nice improvement. Just a typo ☝️ |
Thank you very much, and apologies for the first PR riddled with wrong code! |
[addon-a11y] Fix best practices added in #14736
Issue: in #14736, I added content that had a few issues 😨
This time, let’s get it right and gather feedback from a few more folks before merging. I'd appreciate a thorough review and opinions on these best practices I'm advocating for.