-
Notifications
You must be signed in to change notification settings - Fork 125
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
Editorial: Provide examples of types of menus #2056
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: Valerie Young <[email protected]>
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.
I think this should address more of the most misunderstood aspects of appropriate menu usage and reference the APG. I provided a specific suggestion.
I would love to see this PR move forward. |
imo, the prior suggestions went a bit beyond scope for what I think was needed for this update. So I pulled in some of the suggested updates, but not all.
✅ Deploy Preview for wai-aria ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
ping @mcking65, @cookiecrook , @curtbellew (since you were all signed up to review) to get at least one more reviewer so this can be merged, please. |
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.
approving with minor comments
Co-authored-by: James Craig <[email protected]>
Co-authored-by: James Craig <[email protected]>
Closes #1084
Updated menu content to attempt to help differentiate
menu
andmenubar
, per the request of the original issue.Preview | Diff