Closed (fixed)
Project:
Boldy
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
6 Aug 2012 at 21:35 UTC
Updated:
21 Aug 2012 at 02:31 UTC
After running Functional Accessibility Evaluator 1.1 from The University of Illinois at Urbana-Champaign on the boldy.d7.garethalexander.co.uk demo site.
Under Navigation & Orientation in the Titles (title & h1) section there is:
Fail: 1 page (of 31) with this error
The page should contain no more than two h1 elements.
Warn: 7 pages (of 31) with this warning
The text content of each h1 element should match all or part of the title content.
Warn: 1 page (of 31) with this warning
Each h1 element should have text content exclusive of the alt text of any img elements it contains.
Comments
Comment #1
the_g_bomb commentedSee http://html.cita.illinois.edu/nav/title/ for best practices
Comment #2
the_g_bomb commentedThere's a bit more to this one.
More than one h1 tag. This was down to the portfolio, because the node tpl added a h1 for each node that was output. Changed that to a h2 tag, problem sorted.
h1 content should match all or part if the title.
This was sorted by the above fix for 6 of the pages.
Each h1 should have text content.
This was due to the fact that I switched off the sitename and slogan in the theme settings to allow the logo image to be the only thing in the header bar.
I had to rebuild the site_name and slogan variables in the template file to be able to output a h1 with these values on the homepage with a class element-invisible.
The only thing that remains here is to get the title of the homepage to reflect the site name and slogan rather than saying "Welcome to Boldy D7" as it does at the moment.
Comment #3
the_g_bomb commentedAdded an is_front check in the template.php file and changed the title to be site_name | slogan for the homepage only.