Take a look at this modal specifically built to be a11y a++ and see if we're missing any requirements. https://micromodal.now.sh/