-
Notifications
You must be signed in to change notification settings - Fork 15k
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
docs: add missing MenuItem roles #19329
Conversation
5ab364b
to
3ab95c7
Compare
39399af
to
39f4870
Compare
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.
LGTM
39f4870
to
f1f5b10
Compare
f1f5b10
to
a16d32f
Compare
No Release Notes |
I was unable to backport this PR to "6-0-x" cleanly; |
I have automatically backported this PR to "7-0-x", please check out #19555 |
A maintainer has manually backported this PR to "6-0-x", please check out #19573 |
Description of Change
Fixes #19321.
Adds missing
MenuItem
role typings. Also adds them to the smoke test to prevent future regressions.cc @MarshallOfSound
Checklist
npm test
passesRelease Notes
Notes: none