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
Cleanup & document a11y migration #5833
Conversation
This pull request is automatically deployed with Now. |
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.
Thanks @shilman! Small suggestions
MIGRATION.md
Outdated
addParameters({ a11y: options }); | ||
``` | ||
|
||
You can pass `a11y` parameters at the global level (via `addParameters` imported from `@storybook/react` et al.), and the story level (via the third argument to `.add()`). |
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.
Should this line be "the component level"? (given the example above is global)
Codecov Report
@@ Coverage Diff @@
## next #5833 +/- ##
==========================================
- Coverage 34.31% 34.31% -0.01%
==========================================
Files 648 648
Lines 9461 9463 +2
Branches 1370 1344 -26
==========================================
Hits 3247 3247
- Misses 5596 5598 +2
Partials 618 618
Continue to review full report at Codecov.
|
Cleanup & document a11y migration
I understand that Can we keep the old spelling as a deprecated export? |
Ugh, sorry about that @Hypnosphi. Will fix now! |
Issue: #5832
What I did
withA11Y
towithA11y
for consistencyHow to test
See new story in
official-storybook