Invalid markup in XHTML

[GoogleCodeExporter]

When I try to validate my page (XHTML 1.0 Strict), I get an error:

 Line 17, Column 199: end tag for "link" omitted, but OMITTAG NO was specified 
…608ee6f61356.css,_style,s+css,_color,s+js,_countdown,_jquery,ocountdown,s.css
">✉ 
You may have neglected to close an element, or perhaps you meant to 
"self-close" an element, that is, ending it with "/>" instead of ">". 

This relates to where mod_pagespeed has merged several css files together - 
forming the line:

17.<link rel="stylesheet" type="text/css" media="all" 
href="http://173.236.56.78/themes/enuukGreen/cc.f770833204dec8234e53608ee6f61356
.css,_style,s+css,_color,s+js,_countdown,_jquery,ocountdown,s.css">


To fit in with XHTML, this should have a proper closing tag -ie:
17.<link rel="stylesheet" type="text/css" media="all" 
href="http://173.236.56.78/themes/enuukGreen/cc.f770833204dec8234e53608ee6f61356
.css,_style,s+css,_color,s+js,_countdown,_jquery,ocountdown,s.css"/>


Other issues appear where the module inserts the image dimensions - for example:
<img 
src="http://173.236.56.78/uploaded/img/ce.8645aabfe710d131203e130f259e5c49.site-
logo,g.gif" alt="Logo" width=360 height=123 />

This should have quotes around the width and height values, ie:

<img 
src="http://173.236.56.78/uploaded/img/ce.8645aabfe710d131203e130f259e5c49.site-
logo,g.gif" alt="Logo" width="360" height="123" />

Rich
www.rwapservices.co.uk

Original issue reported on code.google.com by rwap.services on 9 Nov 2010 at 10:57

• Merged into: mod_pagespeed does not handle application/xhtml+xml #4

[GoogleCodeExporter]

This is a very clear bug report.  mod_pagespeed should be examining the doctype 
and, if strict XHTML is coming in, strict XHTML should be going out.

We'll let you know as we progress.

Original comment by jmara...@google.com on 9 Nov 2010 at 2:51

[GoogleCodeExporter]

Actually this is a duplicate, but the description is really good so I'm going 
to copy it into a comment in Issue 4.

Original comment by jmara...@google.com on 9 Nov 2010 at 10:33

• Changed state: Duplicate