Skip to content

docs(menu): expand accessibility guidance for AI codegen - #4072

Merged
williamjstanton merged 2 commits into
Workday:a11yfrom
purvas12:purva-a11y-doc-update-menu
Jul 27, 2026
Merged

docs(menu): expand accessibility guidance for AI codegen#4072
williamjstanton merged 2 commits into
Workday:a11yfrom
purvas12:purva-a11y-doc-update-menu

Conversation

@purvas12

Copy link
Copy Markdown
Collaborator

Summary

Restructure Menu accessibility docs to match the Dialog/FormField pattern with minimum structure, built-in behaviors, requirements, and anti-patterns for codegen.

Release Category

Components


Checklist

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/menu/stories/Menu.mdx

Areas for Feedback? (optional)

  • Code
  • Documentation
  • Testing
  • Codemods

Updated the accessibility section for the Menu component. PLease review the content for accuracy and clarity for AI code generation.

Testing Manually

Review the storybook page for Menu and ensure that the accessibility section renders correctly.

Restructure Menu accessibility docs to match the Dialog/FormField pattern with minimum structure, built-in behaviors, requirements, and anti-patterns for codegen.

Co-authored-by: Cursor <cursoragent@cursor.com>
@purvas12
purvas12 requested a review from williamjstanton July 17, 2026 16:37
@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (3)
  • support
  • prerelease/minor
  • prerelease/major

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 2e42b065-a2a8-41ee-a6fb-e1db6116e98e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cypress

cypress Bot commented Jul 17, 2026

Copy link
Copy Markdown

Workday/canvas-kit    Run #11302

Run Properties:  status check passed Passed #11302  •  git commit 48db6c5ba1 ℹ️: Merge 3d8bc017c2bc145772998775f0786e51d057d88f into e97ce23f43709a2b63ae9c506c93...
Project Workday/canvas-kit
Branch Review purva-a11y-doc-update-menu
Run status status check passed Passed #11302
Run duration 02m 28s
Commit git commit 48db6c5ba1 ℹ️: Merge 3d8bc017c2bc145772998775f0786e51d057d88f into e97ce23f43709a2b63ae9c506c93...
Committer purvas12
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 17
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 809
View all changes introduced in this branch ↗︎
UI Coverage  19.55%
  Untested elements 1537  
  Tested elements 371  
Accessibility  99.44%
  Failed rules  5 critical   5 serious   0 moderate   2 minor
  Failed elements 68  

Comment thread modules/react/menu/stories/Menu.mdx Outdated
Comment thread modules/react/menu/stories/Menu.mdx Outdated
Comment thread modules/react/menu/stories/Menu.mdx Outdated
Comment thread modules/react/menu/stories/Menu.mdx Outdated
Comment thread modules/react/menu/stories/Menu.mdx Outdated
Simplify minimum structure, remove programmatic open without Menu.Target, clarify custom target guidance, and document that mode="multiple" is not accessible with Menu.Item.

Co-authored-by: Cursor <cursoragent@cursor.com>
@purvas12
purvas12 requested a review from williamjstanton July 22, 2026 15:16
@purvas12
purvas12 marked this pull request as ready for review July 22, 2026 15:16
@purvas12
purvas12 requested a review from a team as a code owner July 22, 2026 15:16
@purvas12 purvas12 added documentation Affects documentation ready for review Code is ready for review labels Jul 22, 2026

@williamjstanton williamjstanton left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG!

@williamjstanton
williamjstanton merged commit 1043c2a into Workday:a11y Jul 27, 2026
20 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Affects documentation ready for review Code is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants