RecipeFilter icon indicating copy to clipboard operation
RecipeFilter copied to clipboard

Accessibility Enhancement: Allow Esc to close the filter dialog window

Open charmarkk opened this issue 3 years ago • 3 comments

I primarily use a keyboard to navigate the web, and it would be much simpler for me (particularly on longer recipes or recipe sites, like this one from Just One Cookbook) to use the Esc key to also close the RF dialog window. Is this possible?

charmarkk avatar Mar 02 '22 17:03 charmarkk

That's a good idea! Should it also have aria-live="assertive" for screen readers as well?

sean-public avatar Mar 05 '22 05:03 sean-public

I appreciate your enthusiasm. ☺️ I'd have to do some testing to see how it behaves as is. Very rarely do you want to use an assertive announcement (they're usually for urgent errors or alerts), as they interrupt the flow of the screen reader's default behavior.

In most cases, when a dialog opens, keyboard focus should be set inside it and not leave/exit the dialog until the user closes the dialog. (Here's the ARIA example pattern for some nitty gritty. ☺️)

Very happy to see what I can find out to improve the accessibility of this extension and its forks. ☺️

charmarkk avatar Mar 05 '22 05:03 charmarkk

Making the recipe dialog popup screen reader accessible will probably take some more thinking (it's not easily accessible as is). But adding the keyboard accessible to start with will be good!

charmarkk avatar Mar 14 '22 20:03 charmarkk