Page MenuHomePhabricator

Add more ARIA attributes to ULS cog
Open, LowPublic



It adds aria-haspopup, but those should idealla come with aria-controls and aria-pressed.

Version: master
Severity: minor



Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 1:56 AM
bzimport set Reference to bz50921.
bzimport added a subscriber: Unknown Object (MLST).

I don't understand what this is about. Do you mean "area" instead of "aria", and then still, can you provide a scenario where your enhancement request will make sense? Also, is this related to and if so, should it simply be added there as "WIP criteria" instead of an enhancement on a feature that's not even delivered?

ARIA stands for Accessible Rich Internet Applications and is designed to improve the accessibility of Rich Internet Applications, i.e. to make them more usable for people with disabilities[1].


Getting close to a patch.

I'm removing the aria-haspopup="true" attribute, since that's customarily for a drop-down menu that is a direct child of the element, and not useful for interactive dialogs far away in the DOM. The aria-controls="dialog-id" attribute is sufficient to get the reader to the right place.

That said, once inside the dialog, the contents are inaccessible; the dialog can't be navigated by keyboard because it contains no focusable elements. So that'll probably be the next step.

@MattFitzpatrick: Hi! This task has been assigned to you a while ago. Could you maybe share an update? Do you still plan to work on this task, or do you need any help? If you do not plan to work on this task anymore: Please consider removing yourself as assignee (via Add Action...Assign / Claim in the dropdown menu): That would allow others to work on this (in theory), as others won't think that someone is already working on this. Thanks! :)