Confirm modal should have role=alertdialog (4.1.2) #1404

Blackbaud-ToddRoberts · 2018-01-05T20:16:40Z

Expected behavior

Content container in the Confirm modal should have role="alertdialog"

Actual behavior

Container has role="dialog"

Blackbaud-ToddRoberts added the accessibility label Jan 5, 2018

Blackbaud-ToddRoberts mentioned this issue Jan 5, 2018

Manual accessibility testing: Confirmation dialog #1398

Closed

Blackbaud-MattGregg changed the title ~~Confirm modal should have role=alertdialog~~ Confirm modal should have role=alertdialog (4.1.2) Mar 19, 2018

Blackbaud-SteveBrush added the sky-confirm label Jun 29, 2018

blackbaud-conorwright self-assigned this Jun 29, 2018

blackbaud-conorwright added this to the Block 5 Sprint 2 - 6/29/2018 milestone Jun 29, 2018

blackbaud-conorwright mentioned this issue Jun 29, 2018

added ariaRole input to modals. made confirm modal alertdialog. Added aria-modal=true to modals #1784

Merged

Blackbaud-BobbyEarl modified the milestones: Block 5 Sprint 2 - 6/29/2018, Block 5 Sprint 3 - 7/6/2018 Jul 2, 2018

blackbaud-danbowman modified the milestones: Block 5 Sprint 3 - 7/6/2018, Block 5 Sprint 4 - 7/13/2018 Jul 9, 2018

blackbaud-danbowman modified the milestones: Block 5 Sprint 4 - 7/13/2018, Block 5 Sprint 5 - 7/20/2018 Jul 16, 2018

Blackbaud-MichaelCollier modified the milestones: Block 5 Sprint 5 - 7/20/2018, Block 5 Sprint 6 - 7/27/2018 Jul 23, 2018

Blackbaud-SteveBrush closed this as completed in #1784 Jul 30, 2018

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Confirm modal should have role=alertdialog (4.1.2) #1404

Confirm modal should have role=alertdialog (4.1.2) #1404

Blackbaud-ToddRoberts commented Jan 5, 2018

Confirm modal should have role=alertdialog (4.1.2) #1404

Confirm modal should have role=alertdialog (4.1.2) #1404

Comments

Blackbaud-ToddRoberts commented Jan 5, 2018

Expected behavior

Actual behavior