Problem/Motivation
As a screenreader user, I need helpful ARIA landmarks so I can efficiently use the Drupal administration screens and jump to the correct parts of each page. The Seven theme for administration tasks needs ARIA landmarks to improve its accessibility.
Proposed resolution
Screenshots are attached showing the proposed ARIA landmarks to apply to different sections of the page on Drupal administration screens in the Seven theme.
The first screenshot shows the basic landmarks that should appear on every page. The second one shows internal landmarks on some pages that have multiple areas of tools. The third screenshot shows that the tabs (primary tabs and secondary tabs) should get a navigation landmark.
Remaining tasks
Based on the screenshots attached, the appropriate ARIA attributes should be added to the specified elements of the admin pages. This should be a simple job of adding appropriate markup into Seven theme template files.
User interface changes
None, except for screenreader users who will be able to navigate within a page using landmarks.
API changes
None.
(Thanks to David MacDonald who works with the W3C and attended the Montreal Drupal Accessibility Sprint for recommendations on these landmarks.)
Comment | File | Size | Author |
---|---|---|---|
#13 | seven-aria-landmark-roles-1643410-13.patch | 1.53 KB | mgifford |
#12 | Screen Shot 2013-01-24 at 1.11.20 AM.png | 48.99 KB | mgifford |
#10 | seven-aria-landmark-roles-1643410-10.patch | 1.34 KB | mparker17 |
#4 | seven-aria-landmark-roles-1643410-4.patch | 1.3 KB | mgifford |
#1 | aria-landmarks-seven-tabs.png | 86.02 KB | bowersox |
Comments
Comment #1
bowersox CreditAttribution: bowersox commentedComment #2
JohnAlbinShouldn't these ARIA landmarks be in the default templates too?
Comment #3
bowersox CreditAttribution: bowersox commented@JohnAlbin, I suggest we handle the other themes and the default templates as a separate issue (one has not been opened yet, AFAIK). The reason is that they are actually more complex with many more regions that need landmarks (like sidebars, footer, and such). The Seven theme and administration screens are comparatively simple.
But if you feel otherwise, we could add specs here for all core themes and make this one mega-issue to add ARIA landmarks.
Comment #4
mgiffordLet's see if this gets things moving.
Comment #5
mgiffordforgot to tag.
Comment #6
Anonymous (not verified) CreditAttribution: Anonymous commented+1
Comment #7
mgifford#4: seven-aria-landmark-roles-1643410-4.patch queued for re-testing.
Comment #8
mgiffordtagging.
Comment #10
mparker17The file the patch was affecting has moved to
core/themes/seven/templates/page.tpl.php
. Re-rolling.Comment #11
mgifford#10: seven-aria-landmark-roles-1643410-10.patch queued for re-testing.
Comment #12
mgiffordThis looks great. I just added a visual using https://addons.mozilla.org/en-us/firefox/addon/juicy-studio-accessibilit... and it's going to be a great addition!
Comment #13
mgiffordDrat, remembered when reviewing Bartik that we'd agreed to use
role="contentinfo"
. Adding that now. Back to review.Comment #14
mparker17Cool, thanks Mike! :)
I'll definitely have to use that extension in the future!
Comment #15
mparker17Assigning to me to review.
Comment #16
mparker17Sorry about the delay in getting this RTBC'd.
Everything's looking good!
Comment #17
catchLooks sensible to me. Committed/pushed to 8.x.
Comment #18
mgiffordGreat!
Comment #19
mparker17Thanks!
Comment #20
mgiffordAdding link to #1913692: Convert Seven to HTML5 - not really a child issue, but related. Not sure why it wasn't done before.
Comment #21.0
(not verified) CreditAttribution: commentedAdded additional information about tabs.