#2056: fckdialog.html is invalid xhtml
----------------------+-----------------------------------------------------
Reporter: zagy | Owner: alfonsoml
Type: Bug | Status: new
Priority: Normal | Milestone:
Component: General | Version: FCKeditor 2.6 Beta
Resolution: | Keywords: Confirmed Review?
----------------------+-----------------------------------------------------
Comment (by zagy):
Replying to [comment:4 alfonsoml]:
> That patch sounds too aggressive (and it seems to have some error as it
was escaping some html inside javascript)
>
> The proposed patch probably should be enough, as the dialogs mixing html
and javascript did work previously. At the same time it does fix a
possible problem in the link dialog as it wasn't using the same system to
generate the upload iframe.
They might "work" if the parser is lazy enough to let the invalid things
pass. Zope doesn't do that which micht not be of your concern. But the
document has an XHTML *strict* document type and doesn't validate. That is
not just because of closing </foo> notes in HTML/Javascript/document.write
foo. Have you had a look at the URL above? The first thing which actually
breaks is
Line 90, Column 25: character "&" is the first character of a delimiter
but occurred as data.
if ( FCKBrowserInfo.IsIE && !FCKBrowserInfo.IsIE7 )
Unquoted & in XML is just a syntax error. What in fact might help is
adding XML comments inside the <script> tags to avoid parsing. But I don't
really see a reason to not put JS into JS files.
--
Ticket URL: <http://dev.fckeditor.net/ticket/2056#comment:5>
FCKeditor <http://www.fckeditor.net>
The text editor for Internet
-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference
Don't miss this year's exciting event. There's still time to save $100.
Use priority code J8TL2D2.
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac
