[jira] Commented: (FOR-592) Pelt skin not HTML 4.01 compliant

["Gavin (JIRA)" <ji...@apache.org>]

    [ http://issues.apache.org/jira/browse/FOR-592?page=comments#action_12317331 ] 

Gavin commented on FOR-592:
---------------------------

I agree, I am a strong believer in validation and validate any sites I build to XHTML 1.1 and CSS2 where possible (not possible = client un-wavering or not caring).

I would like to help if I can with various parts of Forrest , starting with the little things.
How would I go about helping with this little validation matter?

Unless Manuel is volunteering :)

Gav...

> Pelt skin not HTML 4.01 compliant
> ---------------------------------
>
>          Key: FOR-592
>          URL: http://issues.apache.org/jira/browse/FOR-592
>      Project: Forrest
>         Type: Bug
>   Components: Skins (general issues)
>     Versions: 0.7
>     Reporter: Manuel Mall

>
> Just go to the Forrest home page and click the HTML 4.01 Compliance button. The page doesn't validate. Other sites, e.g. Apache FOP, created with the same skin have the same problem. One issue I could identify is that in the site2xhtml.xsl there are tags like: 
>     <input type="submit" value="Search" name="Search" i18n:attr="value"/>. 
> The 'i18n:attr="value"' bit creates a non HTML 4.01 compliant attribute. Other problems are related to Javascript not being escaped correctly, e.g.  
>      document.write("<text>Last Published:</text> " + document.lastModified);
> should be written as
>    document.write("<text>Last Published:<\/text> " + document.lastModified);
> I assume a requirement of being allowed to put the W3C HTML 4.01 Compliant logo on the site is that the pages correctly validate. Pages based on this skin don't validate but do carry the logo. Sounds like a problem to me.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira