-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
accessibility popover guidance #9024
Conversation
this PR is replacing whatwg#8791 due to the changes to the related popovertarget attributes. i've added more examples and incorporated feedback that was provided from the review of the original PR.
remove 'recommended' from note
fix typo in closing tag order
add missing id to sub-navigation popover example. one last example showing how to use popover=manual to reveal a popover as a status message (e.g., a toast).
I had been asked in an offline conversation if i could get a status message / toast-like example into the spec for popover=manual. Hence the most recent example added. If anyone thinks this is adding too much, or if more details are necessary to accompany this example, please let me know. |
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 modulo @josepharhar's style comments.
fix incorrect attribute, re catch from @mfreed7
resolves feedback from @josepharhar
thank you @josepharhar @mfreed7. changes have been pushed |
after speaking with @aleventhal, a note about popovers/live regions has been added after the code examples.
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 @scottaohara! I've got mainly nits, one more substantive comment.
this addresses the feedback from @annevk's review.
thank you @annevk. have incorporated your feedback |
Thanks @scottaohara. I plan on merging this Monday. If there's any remaining nits at that point I'll attempt to fix them. |
this PR is replacing #8791 due to the changes to the related popovertarget attributes. Was much easier to start a new PR than resolve the diffs.
i've added more examples and incorporated feedback that was provided from the review of the original PR.
added the labels and reviewers from the original PR.
Thanks!
/popover.html ( diff )