canvas-kit
canvas-kit copied to clipboard
docs: Migrating accessibility guidelines for Expandable & Menu
Summary
As part of making accessibility a "first class citizen" in Canvas docs, we will be pulling any technical information for developers off of the design site, and into the Canvas repository.
Expandable and Menu are the next set of components.
Release Category
Documentation
Checklist
- [ ] MDX documentation adheres to Canvas Kit's Documentation Guidelines
- [ ] Label
ready for reviewhas been added to PR
For the Reviewer
- [ ] PR title is short and descriptive
- [ ] PR summary describes the change (Fixes/Resolves linked correctly)
- [ ] PR Release Notes describes additional information useful to call out in a release message or removed if not applicable
- [ ] Breaking Changes provides useful information to upgrade to this code or removed if not applicable
Where Should the Reviewer Start?
modules/react/expandable/stories/Expandable.mdx
modules/react/menu/stories/Menu.mdx
I also updated the examples using the new Avatar component and the isDecorative prop.
Areas for Feedback? (optional)
- [ ] Code
- [X] Documentation
- [ ] Testing
- [ ] Codemods
Testing Manually
Screenshots or GIFs (if applicable)
Thank You Gif (optional)
Workday/canvas-kit
Run #9967
Run Properties:
Passed #9967 •
d3f30abd66 ℹ️: Merge a0ec74114d66cf2fd7a1db61865e1eaec25d1e7a into 6589a979c0a56ef57b3fefb0057e...
| Project |
Workday/canvas-kit
|
| Branch Review |
william-docs-expandable-and-menu
|
| Run status |
|
| Run duration | 02m 18s |
| Commit |
|
| Committer | William Stanton |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
|
|
0
|
|
|
0
|
|
|
86
|
|
|
0
|
|
|
862
|
| View all changes introduced in this branch ↗︎ | |
UI Coverage
19.4%
|
|
|---|---|
|
|
1535
|
|
|
367
|
Accessibility
99.33%
|
|
|---|---|
|
|
6 critical
5 serious
0 moderate
2 minor
|
|
|
77
|