Page MenuHomePhabricator

Add missing keyboard navigation events to popover component
Closed, ResolvedPublic5 Estimated Story Points

Description

Users should be able to dismiss popovers using their keyboard by performing any of the following actions:

  • pressing enteragain, in case the element that triggered the popover is still focused
  • pressing esc, while the popover is open
  • pressing tab to remove the focus from the trigger element and place it on the next focusable element on the page.

ACS:

  • Users are able to not only enable, but also dismiss the popover using their keyboard.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Removing the wdwb-tech tag after realizing that this is rather a user-facing, product task.

Addshore moved this task from Triaged Low (0-50) to Product Realm on the [DEPRECATED] wdwb-tech board.
Addshore subscribed.

Removing the wdwb-tech tag after realizing that this is rather a user-facing, product task.

Added it back, but moved to "product realm" and I'll remove it from our tech prio backlog for that reason too

Addshore set the point value for this task to 5.Jun 23 2021, 10:50 AM

What about pressing Space? That key also toggles buttons, and at least in the storybook, it opens the popover, but with the current version of the pull request it doesn’t close it again.

Agh! My bad, I overlooked that. So pressing space, like with enter, should dismiss the popover when the focus is still on its trigger.