Opened 11 years ago
Closed 11 years ago
#26950 closed defect (bug) (fixed)
Twenty Eleven broken screen reader link
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.9 | Priority: | normal |
Severity: | normal | Version: | 3.9 |
Component: | Bundled Theme | Keywords: | has-patch |
Focuses: | accessibility | Cc: |
Description
The Twenty Eleven theme adds two skip links for accessibility: "skip to primary content" and "skip to secondary content". The problem is, the "secondary content" does not always exist. All singular posts and pages do not have the linked-to secondary content section, rendering the link broken on all of those pages.
Since none of the default themes since Twenty Eleven have had a second skip link, it seems like removing the secondary skip link entirely would be the best course of action.
Attachments (2)
Change History (8)
#5
in reply to:
↑ 4
@
11 years ago
Replying to johnbillion:
The
title
attribute on these links are also redundant.
Nearly all title attributes are at least conceptually redundant. These are literally redundant! As it happens the original 26950.patch is unusable, I somehow dropped the !
modifier to is_singular()
, so 26950.no-title.patch is the only usable patch at this point.
Removing the link would be easy, but I don't like the idea of removing established functionality. It's not so difficult to make the link output conditional on
!is_singular()
.