Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

Modal container should have aria-modal=true (4.1.2) #1403

Closed
Blackbaud-ToddRoberts opened this issue Jan 5, 2018 · 1 comment
Closed

Modal container should have aria-modal=true (4.1.2) #1403

Blackbaud-ToddRoberts opened this issue Jan 5, 2018 · 1 comment
Assignees
@blackbaud-conorwright
Labels
accessibility

Comments

@Blackbaud-ToddRoberts
Copy link
Contributor

Expected behavior

The content container in Modal should have:
role="dialog"
aria-modal="true"

Actual behavior

only has role="dialog"
Closed
@Blackbaud-MattGregg Blackbaud-MattGregg changed the title Modal container should have aria-modal=true Modal container should have aria-modal=true (4.1.2) Mar 19, 2018
Merged
@blackbaud-conorwright blackbaud-conorwright self-assigned this Jun 29, 2018
@Blackbaud-MattGregg
Copy link
Contributor

Tested regular and full page modals today. Looks fixed to me. Closing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@blackbaud-conorwright blackbaud-conorwright

Labels
accessibility
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
8 participants
@Blackbaud-BobbyEarl @blackbaud-johnly @Blackbaud-SteveBrush @blackbaud-danbowman @Blackbaud-ToddRoberts @Blackbaud-MattGregg @blackbaud-conorwright @Blackbaud-MichaelCollier