search

bmlt-enabled / crouton

BMLT Tabbed-based view for Wordpress https://wordpress.org/plugins/crouton/ and any website. https://github.com/bmlt-enabled/crouton/tree/master/croutonjs
GNU General Public License v2.0
5 stars 14 forks source link

Accessibility Issues with Meeting Details Page Modal #472

Open klgrimley opened 7 months ago

klgrimley commented 7 months ago

Describe the bug 1) Modal does not capture focus when opened. Forced to tab through entire page to entire page. This is confusing as you can't see where focus currently is as modal is open.

2) No way to close modal with keyboard. Neither the "x" in the top right corner nor the close button in the bottom right receives focus.

3) Modal can't be opened via keyboard navigation

otrok7 commented 7 months ago

Hi Kevin, please test 3.18.12 to see if it fixes these issue... it will be out later today

klgrimley commented 6 months ago

Sorry, didn't see the email notification about the comment. Overall it's an improvement. The basics of all the points are covered.

Point 1 is resolved. With the caveat that it's the map "menu" that's initially focused on. Typically the first focusable/interactive element would be the close "X" in the top right corner

Point 2 The "X" never does get focus. While there is a way to close the modal, the bottom right close button, the top right is where a user would expect to have initial focus. There's also not a need for two close options and could be considered an issue although they're probably far enough apart.

Point 3 is entirely resolved.

I know some of this is nit picky. But I just really don't want to see one of NA sites end up with a law suit. It would be unlikely in the States. But I know Europe is in general a few years a head in regards to accessibility enforcement.