Validation Errors for language and border attributes

Came here looking for this issue. These errors are seen when using a strict doctype.

line 780 column 79 - Error: document type does not allow element "a" here; missing one of "p", "h1", "h2", "h3", "h4", "h5", "h6", "div", "pre", "address", "fieldset", "ins", "del" start-tag
line 781 column 28 - Error: there is no attribute "border"

This module should output xhtml valid code.

The offending lines:

778 <script language="JavaScript" type="text/javascript"><!--
779 if(navigator.appVersion.indexOf('MSIE')>=0)document.write(unescape('%3C')+'\!-'+'-')
780 //--></script><noscript><a href="http://www.omniture.com" title="Web Analytics"><img src="/8143892"
781 height="1" width="1" border="0" alt="" /></a></noscript><!--/DO NOT REMOVE/-->

+1 for all the above

It is worth keeping in mind that HTML validation tools are just that, tools, and achieving fully validated HTML is not more important than delivering intelligent, well thought out HTML.

That said, D6's default theme comes with DOCTYPE strict, rather than the more past/future friendly HTML5, so my suggestion would be:

• Remove the language attribute
• Remove the border attribute
• Add a div to the inside of the noscript

The language attribute is depreciated

The border attribute is depreciated (an OLD safeguard to stop the pretty blue border from appearing on images within an anchor tag, which Drupal's default.css (img { border: 0; }) takes care of).

Validation is important to some, so if only to placate the people who like to add the "This site validates" image on their site, add the div so the noscript error disappears - I have NEVER seen a logical explanation for why the noscript thinks it needs this, but while we have DOCTYPE strict in D6, and the validating tools expect it.

Better statuses. I'm not really in a position to test/review this, but if a consensus forms among users I would be happy to commit.

The attached patch should validate strict