The skip nav anchor link points to #main-content, but there's no element with that id on the page. This is an accessibility issue, the link is needed for screen readers and users navigating by keyboard.

CommentFileSizeAuthor
#1 skip-nav-link-broken-1166276.patch1.28 KBsarahjean

Comments

sarahjean’s picture

sarahjean commented
StatusFileSize
new skip-nav-link-broken-1166276.patch1.28 KB

Here's a patch that I have made that adds a target for the skip nav link to the base theme page.tpl.php and the page.tpl.php inside of starterkit and startkit-xhtml. Meant to follow up sooner, this is my first patch so hopefully it will work properly.

sarahjean’s picture

sarahjean commented
Title: Skip nav link broken » Skip nav link broken, should point to the main content
Status: Active » Needs review

Changing the status and updating the title to make it a little more descriptive.

Since there isn't a target for the link currently, it won't actually skip the nav if you try to use it. Hopefully someone can take a look at this, the patch above is what I have done in my subtheme to fix the link.

fubhy’s picture

fubhy commented
Status: Needs review » Active
mgifford’s picture

mgifford
he/him
Primary language English
commented

I don't know if this is an issue or not (haven't looked yet), but this patch no longer applies as the page.tpl.php only is occurring in alpha/templates/page.tpl.php in the git repository.

sarahjean’s picture

sarahjean commented

This seems to have been fixed in 7.3 at least.

cellar door’s picture

cellar door commented
Status: Active » Closed (fixed)

Closing out as fixed as 7.2.x is no longer the recommended version of Omega