-
Notifications
You must be signed in to change notification settings - Fork 153
fix(storybook): add a11y checker back, react/wc #8344
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
base: main
Are you sure you want to change the base?
fix(storybook): add a11y checker back, react/wc #8344
Conversation
✅ Deploy Preview for ibm-products-web-components ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for carbon-for-ibm-products ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8344 +/- ##
==========================================
+ Coverage 79.38% 83.91% +4.53%
==========================================
Files 474 394 -80
Lines 20340 17944 -2396
Branches 5006 4165 -841
==========================================
- Hits 16146 15057 -1089
+ Misses 4194 2887 -1307
🚀 New features to boost your workflow:
|
|
|
This PR will add the accessibility checker addon back to our storybook, since we had to remove it when upgrading to storybook v9. The checker package now supports v9 via
[email protected]. There are some already documented issues when violations are present (see more in this PR), but otherwise the addon tab is back.What did you change?
How did you test and verify your work?
Manually running react and web component storybooks and verifying the
Accessibilitytab is presentPR Checklist
As the author of this PR, before marking ready for review, confirm you:
More details can be found in the pull request section of
our contributing docs.