In page.tpl.php:

$grep -n id\=\"menu\" page.tpl.php
43: <table border="0" cellpadding="0" cellspacing="0" id="menu">
46:    <td id="menu">

Creates an Error with http://validator.w3.org

CommentFileSizeAuthor
#3 wabi.patch2.43 KBJeff Burnz

Comments

Jeff Burnz’s picture

Jeff Burnz commented
Assigned: Unassigned » Jeff Burnz

Shocking but true, thanks for pointing this out, any chance of a patch?

tbals’s picture

tbals commented

Maybe you could rename the id in Line 46, I cant find any uses in style.css.
On my side I temporary renamed to menu-m and cant see any differences in the browser output.

Jeff Burnz’s picture

Jeff Burnz commented
Status: Active » Needs review
StatusFileSize
new wabi.patch2.43 KB

Heres a patch - chagnes the id=menu to id=menu-links, also includes fixes for the following other issues:

http://drupal.org/node/313621 - document type does not allow element "ul" here
http://drupal.org/node/313617 - some td's not closed

tbals’s picture

tbals commented

Looks good - now the w3c validator says for startpage "This document was successfully checked as XHTML 1.0 Strict!"
Thx for quick patch.

Regards
Thomas

peterx’s picture

peterx commented

Would love to see a formal D6 update so I can approve the theme on http://d-theme.com/wabi.

Jeff Burnz’s picture

Jeff Burnz commented
Status: Needs review » Patch (to be ported)