Paths

Table of Contentst

Differential D19595

Add more accessibility labels for screen readers
ClosedPublic
Actions

Authored by epriestley on Aug 17 2018, 5:56 PM.

Tags

None

Referenced Files

	F18932313: D19595.id46844.diff
	Mon, Nov 10, 7:05 AM

	F18876879: D19595.id.diff
	Thu, Nov 6, 4:42 AM

	F18864032: D19595.diff
	Mon, Nov 3, 2:17 AM

	F18825735: D19595.id.diff
	Oct 23 2025, 11:56 PM

	F18818915: D19595.id46850.diff
	Oct 22 2025, 3:12 AM

	F18810322: D19595.id.diff
	Oct 19 2025, 9:04 PM

	F18805013: D19595.diff
	Oct 18 2025, 1:09 PM

	F18642614: D19595.diff
	Sep 19 2025, 12:20 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T13164: Plans: 2018 Week 31 - 33 Bonus Content

Commits

rP75a5dd8d8cc0: Add more accessibility labels for screen readers

Summary

Depends on D19594. See PHI823. Ref T13164.

Add a label for the "X" button in comment areas, like "Remove Action: Change Subscribers".
Add a label for the floating header display options menu in Differential.
Add role="button" to PHUIButtonView objects that we render with an <a ...> tag.

Test Plan

Viewed a revision with ?__aural__=true:

Saw "Remove Action: ..." label.
Saw "Display Options" label.
Used inspector to verify that some <a class="button" ...> now have <a class="button" role="button" ...>. This isn't exhaustive, but at least improves things. A specific example is the "edit", "reply", etc., actions on inline comments.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Aug 17 2018, 5:56 PM

Harbormaster completed remote builds in B20643: Diff 46844.Aug 17 2018, 5:58 PM

epriestley requested review of this revision.Aug 17 2018, 5:58 PM

amckinley accepted this revision.Aug 17 2018, 6:50 PM

This revision is now accepted and ready to land.Aug 17 2018, 6:50 PM

Closed by commit rP75a5dd8d8cc0: Add more accessibility labels for screen readers (authored by epriestley). · Explain WhyAug 17 2018, 8:31 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

src/

applications/

differential/

view/

DifferentialChangesetListView.php

1 line

transactions/

view/

PhabricatorApplicationTransactionCommentView.php

6 lines

view/

phui/

PHUIButtonView.php

8 lines

webroot/

rsrc/

js/

application/

diff/

DiffChangesetList.js

7 lines

transactions/

behavior-comment-actions.js

5 lines

phuix/

PHUIXButtonView.js

25 lines

Diff 46850

resources/celerity/map.php

Loading...

src/applications/differential/view/DifferentialChangesetListView.php

Loading...

src/applications/transactions/view/PhabricatorApplicationTransactionCommentView.php

Loading...

src/view/phui/PHUIButtonView.php

Loading...

webroot/rsrc/js/application/diff/DiffChangesetList.js

Loading...

webroot/rsrc/js/application/transactions/behavior-comment-actions.js

Loading...

webroot/rsrc/js/phuix/PHUIXButtonView.js

Loading...