﻿id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
2630	Empty span tags not removed when using Placeholder plugin	Niels Noorlander		"When using the placeholder plugin, which is part of the standard FCK download, empty XHTML-style span tags no longer get removed.

For example, enter the following HTML in de source view:


{{{
<html dir=""ltr"">
    <head>
        <title>Test</title>
    </head>
    <body>
    	<span style=""color: white"" />
    </body>
</html>
}}}

Switch to WYSIWYG-mode and back to the source. Without the placeholder plugin, the span will be removed. With the plugin enabled, the span will stay, causing unwanted behaviour because of a IE6/7 rendering bug for XHTML span tags.

After some debugging, I've discovered this problem is easily fixed by changing one line in the plugin:

Original code:
{{{
    FCKXHtml._AppendChildNodes( node, htmlNode, false ) ;
}}}

Fix:
{{{
    node = FCKXHtml._AppendChildNodes( node, htmlNode, false ) ;
}}}

And maybe even better;
{{{
node = FCKXHtml._AppendChildNodes( node, htmlNode, Boolean( FCKListsLib.NonEmptyBlockElements[ sNodeName ]) ) ;

}}}

Ofcourse this fix is not critical to standard FCK functionality, but the placeholder plugin is an interesting example for FCK customization and would be good to fix it.

Kind regards,

Niels Noorlander.
"	Bug	closed	Normal		Core : Output Data	3.0	fixed	HasPatch	
