A link named "Skip to content" links to an anchor named "content" which does not exist. There is a class called content, but not a name.

CommentFileSizeAuthor
#1 remove_content_link.patch409 bytesgreggles

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Active » Needs review
StatusFileSize
new remove_content_link.patch409 bytes

I'm not sure if this needs fixing really since it's in a hidden span anyway, but I removed it on mysite and provide this patch.

factoryjoe’s picture

factoryjoe commented

Please do not remove the link. Instead, add the necessary ID, whatever you want to use.

The link is there for accessibility purposes as well as for getting to the content from a cell-phone or other device that doesn't support CSS-2.

factoryjoe’s picture

factoryjoe commented

So basically, -1.

watermark’s picture

watermark commented
Status: Needs review » Active

Here is a solution:

In page.tpl.php change the Skip link to this (originally hidden, this is not compatible with screen readers and the like)

<body <?php print theme("onload_attribute"); ?>>
<a href="#content" class="skip" title="Skip the site navigation to go directly to the content">Skip to content</a>
<?php if ($site_slogan) : ?>

Further down in page.tpl.php add the anchor link:

<!-- start main content -->
<span style="width: 100%;"><a name="content"></a></span>
<?php print($content) ?>
<!-- end main content -->

In one of the style sheets (I used layout.css) add the following:

.skip { 
  position: absolute; 
  left: -1000em; 
  width: 20em;
  }

If you want to read about the subject, then go here: http://www.jimthatcher.com/skipnav.htm

Cheers,
Mark

Mad Maks’s picture

Mad Maks commented
Assigned: Unassigned » Mad Maks
Status: Active » Fixed

fixed in the cvs version

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)