-
Notifications
You must be signed in to change notification settings - Fork 86
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
Non-autofocused Popover not reachable by keyboard #8702
Comments
Just to clarify: this has been implemented by #7609 according to the specification from #7607. Especially, instead of "moving it back to the target element after the last tab stop inside the overlay" we move focus to the next element in the DOM that comes after the popover target (and I believe this is how it would also work with native popover). |
Ah, indeed, I think the reason I thought it hadn't been is that it doesn't work with the "interactive tooltip" sample in the docs, but I suppose that is because the popover actually closes on blur when you tab onwards from it? (What actually happens there is when tabbing while focus is in the field is that the popover closes but focus doesn't move anywhere – presumably due to this mechanism.) |
Yes, I think the problem in that example is that it closes due to the focus trigger. I'll take a look if we can fix that. |
I actually already created this ticket: vaadin/docs#4147 |
Actually, I found a bug with the Tab logic of the popover: it checks So this is somewhat similar to #8703 where wrong element is used. I'll work on a fix to also check for |
What is the problem?
A Popover that is not modal or otherwise configured to automatically steal focus is inherently inaccessible by keyboard (unless some custom means of moving focus to it is provided).
A couple of solution options come to mind:
popover
(and/or<dialog>
) and moving them from the dedicated overlay section at the end of the DOM to be immediate siblings of the target element, whereby the overlay's first tab stop will natively follow the target element.Browsers
Screen Readers
The text was updated successfully, but these errors were encountered: