Closed (fixed)
Project:
Dropshadow
Version:
6.x-1.1-beta1
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
25 Feb 2009 at 04:51 UTC
Updated:
27 Mar 2009 at 04:20 UTC
The html element is missing language. Direction is also recommended. Current:
<html xmlns="http://www.w3.org/1999/xhtml">
Required
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="<?php print $language->language; ?>" lang="<?php print $language->language; ?>">
Recommended
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="<?php print $language->language; ?>" lang="<?php print $language->language; ?>" dir="<?php print $language->dir; ?>">
Comments
Comment #1
peterx commentedThe default style sheet language must be specified when style attributes are used. The following line appears twice. The style should be replaced by a class.
<div style="clear: both;">When these changes are made, the theme should validate against WCAG and other standards.
Comment #2
peterx commentedThis next problem is a bit trickier. The following content title has a link back to itself and a conflict with another link, creating a WAI AA problem. The link back to the current page makes no sense so dropping the link would work. The link conflicts with another link having the same text but a different href. Adding a title to explain the link would make the link unique to a person reading the screen through a screen reader. I just cannot think of what title you would add to a link back to the same page.
<div class="contenttitle"><h1><a href="/dropshadow">Dropshadow</a></h1>Comment #3
peterx commentedBeta 1 is valid XHTML 1.0 Strict at http://d-theme.com/dropshadow.