Page MenuHomePhabricator

Accessibility of headers broken due to accessibility enhancements
Closed, ResolvedPublic

Description

The H2 headers in MF were recently enhanced to make their collapsibility more accessible (by Hoo ???). Because of this they now carry role="button". This will break accessibility because it will override the default role of "heading" of the H2, which is not desirable.

It should be role="heading button"


Version: unspecified
Severity: normal

Details

Reference
bz68371

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:38 AM
bzimport set Reference to bz68371.
bzimport added a subscriber: Unknown Object (MLST).
TheDJ created this task.Jul 22 2014, 11:17 AM

bingle-admin wrote:

Prioritization and scheduling of this bug is tracked on Trello card https://trello.com/c/lIjuUOf0

Change 148363 had a related patch set uploaded by Hoo man:
Make sure headings with role button also have the heading role

https://gerrit.wikimedia.org/r/148363

Hoo man and TheDJ how can we get this patch wrapped up (I see theDJ -1ed it)?

Releasing into wild.

Change 148363 abandoned by Jdlrobson:
Make sure headings with role button also have the heading role

Reason:
Closing due to lack of activity. Please reopen this and resubmit it if you are still keen to fix this patch (someone else should feel free to fork it and fix it up).

https://gerrit.wikimedia.org/r/148363

Change 148363 restored by Bmansurov:
Make sure headings with role button also have the heading role

Reason:
adding a new patch

https://gerrit.wikimedia.org/r/148363

Change 148363 merged by jenkins-bot:
Remove role 'button'

https://gerrit.wikimedia.org/r/148363