A11y/modals - Githubissues

pixelandtonic / garnishjs

UI toolkit by Pixel & Tonic

MIT License

39 stars 5 forks source link

A11y/modals #16

Open gcamacho079 opened 2 years ago

gcamacho079 commented 2 years ago

Description

Aims to fix a few accessibility issues with modals, including the following:

Activating a focus trap on the modal
Setting aria-hidden="true" on all sibling elements in the body (not including style or script tags)
Add triggerElement setting to store default element to refocus on close

This is being worked on consecutively with updates in the CMS repo

Related issues

CMS-009, CMS-010, and CMS-014 in issue tracker

gcamacho079 commented 2 years ago

@brandonkelly this is nearly there, but I'd like to test on an instance (if there is one) where two modals are visible at once to ensure the ariaHide function is working properly.