Add role=dialog to all dialogs
ClosedPublic
Actions

Authored by chad on May 31 2016, 7:20 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Jul 5 2026, 11:46 AM

	Unknown Object (File)
	Jul 4 2026, 9:49 AM

	Unknown Object (File)
	Jul 2 2026, 8:41 PM

	Unknown Object (File)
	Jun 29 2026, 11:41 PM

	Unknown Object (File)
	Jun 18 2026, 7:33 PM

	Unknown Object (File)
	Mar 6 2026, 8:18 PM

	Unknown Object (File)
	Feb 11 2026, 1:39 PM

	Unknown Object (File)
	Feb 10 2026, 7:52 PM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Commits

rPb23c85b16915: Add role=dialog to all dialogs

Summary

Seen some complaints about usability here, adding role=dialog to improve when these trigger.

Test Plan

Turn on Voiceover, tab over to log out link, here proper dialog title and text before highlighted submit button.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad updated this revision to Diff 38495.May 31 2016, 7:20 PM

chad retitled this revision from to Add role=dialog to all dialogs.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added a reviewer: epriestley.

Herald added a subscriber: Korvin. · View Herald TranscriptMay 31 2016, 7:20 PM

epriestley accepted this revision.May 31 2016, 7:20 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.May 31 2016, 7:20 PM

Closed by commit rPb23c85b16915: Add role=dialog to all dialogs (authored by chad, committed by chad). · Explain WhyMay 31 2016, 7:21 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

view/

AphrontDialogView.php

1 line

Diff 38496

View Options

src/view/AphrontDialogView.php

Add role=dialog to all dialogsClosedPublicActions