The Toolbar has been Frozen, textarea uneditable

[realvalkyrie]

the code as follows:

<B><FONT face=Tahoma size=2><SPAN style="FONT-WEIGHT: bold; FONT-SIZE: 10pt; FONT-FAMILY: Tahoma">De:</SPAN></FONT></B><FONT size=2><SPAN style="FONT-SIZE: 10pt; FONT-FAMILY: Tahoma"> Devz <BR><B><SPAN style="FONT-WEIGHT: bold">Enviado el:</SPAN></B> MartZ<BR><B><SPAN style="FONT-WEIGHT: bold">Para:</SPAN></B> Mary Lua<BR><B><SPAN style="FONT-WEIGHT: bold">Asunto:</SPAN></B> RV: Documents</SPAN></FONT><FONTFACE="MS PGothic?><BR>Dear Devito,</SPAN></FONT><FONT face="Times New Roman" size=3><SPAN style="FONT-SIZE: 12pt"><BR>ADSDF</SPAN></FONT>

[realvalkyrie]

if replace the invalid html tag "FONTFACE" to "FONT FACE" it's work perfect

[fredck]

Confirmed in IE only. Works well with FF.

Simply paste the provided HTML in the source view and move back to wysiwyg. The toolbar gets stuck, as well as the editor editing.

[m.nguyen]

if replace the invalid html tag "FONTFACE" to "FONT FACE" it's work perfect

We haven't FONTFACE tag, we have a FONT tag and FACE attribute. ​http://www.w3.org/TR/html4/index/elements.html

Works well with FF

In this case, FF (Safari, Chrome) ignore invalid tags. You can try "XYZ" tag or "Nguyen" tag it still display well.

[realvalkyrie]

fredck's description is exactly right。 3Q 4 All reply

[Saare]

The line 'window.parent.CKEDITOR._contentDomReadyeditor1?( window ); ' is also appended.
I found out it happens because of the unclosed tag produced, and not because of an invalid tag.
'<FONTFACE="MS PGothic?>' - instead of quote mark there is a question mark, which breaks the HTML. here's an example:

<FONTFACE="MS PGothic?><FONT face="Times New Roman" size=3>"ADSDF

the parser is parsing it as:

<FONTFACE="
MS PGothic?><FONT face=
"
Times New Roman"
 size=3>
"
ADSDF

so it looks like an unclosed tag.

[Saare]

With this patch, paste the following into source:

<FONTFACE="MS PGothic?><FONT face="Times New Roman" size=3>"ADSDF

Switch to WYSIWYG mode. there is no content in the editor.
Switch back to Source mode. there is a code inside.
Switch again to WYSIWYG mode. now there is a content in the editor.

[fredck]

I'm not going on a detailed review for now, but the patch looks plain wrong. Deferring.

[fredck]

I've just spent 2 hours on reviewing and research on this issue. We must make it in a way that the invalid tag will be simply ignored, instead of trying to use it somehow... of course, without affecting performance.

I'll be proposing something.

[fredck]

While testing this patch, I haven't noticed any performance impact.

[fredck]

Fixed with [5740].