Minify_HTML breaks DOM interpretation with attributes.

[Google Code Exporter]

Minify_HTML::process

Putting attribute list on new line breaks the DOM interpretation, as the tag 
and attribute name are seen together.

To fix it, I tried adding a space next-to newline.

(file: lib/Minify/HTML.php line: ~139)
$this->_html = preg_replace('/(<[a-z\\-]+)\\s+([^>]+>)/i', "$1\n $2", 
$this->_html);

Original issue reported on code.google.com by [e-mail] on 17 Jun 2011 at 11:24

[Google Code Exporter]

Can you give an example of markup it breaks?

Original comment by [e-mail] on 17 Jun 2011 at 4:26

[Google Code Exporter]

I had following snippet in a template file and was trying to minify and merge 
it on the html page. 

<div class="toolbar"> View:</div>

The output was:
<divclass="toolbar"> View: </div>

Hope it helps.

Original comment by [e-mail] on 21 Jun 2011 at 5:07

[Google Code Exporter]

I can't reproduce this bug. The output I get has a newline between "<div" and 
"class=". Can you give me another example?

Original comment by [e-mail] on 23 Jun 2011 at 7:28

[Google Code Exporter]

Try the following:

<?php

set_include_path(dirname(__FILE__). '/libs/min/lib' . PATH_SEPARATOR . 
get_include_path());
include_once 'Minify/HTML.php';

$input = '<div class="aClass">aContent</div>';

echo Minify_HTML::minify($input);
?>

Output was:
<div
class="aClass">aContent</div>

which firefox 4.x interpreted as <divclass="aClass"... and missed applying the 
style.

Original comment by [e-mail] on 24 Jun 2011 at 4:57

[Google Code Exporter]

It's perfectly valid to use "\n" rather than " " as whitespace in a tag, and 
browsers should have no problem parsing it. FF5/XP and FF4/OSX both do fine for 
me rendering these pages, which are processed by Minify_HTML:

http://mrclay.org/min/builder/
http://fawn.ifas.ufl.edu/mobile/260

Can you create a public demo page so others can verify this is indeed a FF4 
parsing bug and not some kind of authoring error?

Original comment by [e-mail] on 24 Jun 2011 at 9:54

• Changed state: NeedInfo
• Added labels: Priority-Low
• Removed labels: Priority-Medium

[Google Code Exporter]

Original comment by [e-mail] on 3 Sep 2011 at 11:11

• Changed state: Invalid