#3801: IE: Comments at the start of the document get lost
----------------------------------+-----------------------------------------
Reporter: fredck | Owner: tobiasz.cudnik
Type: Bug | Status: assigned
Priority: Normal | Milestone: CKEditor 3.0
Component: General | Version: SVN (CKEditor)
Keywords: Confirmed IE Review- |
----------------------------------+-----------------------------------------
Comment(by tobiasz.cudnik):
No problem with the comment, but i don't understand which "head
whitespace" you had on mind. We can't use whitespace instead of "a" (or
other char) because it doesn't have same result. If there are whitespaces
before the HTML comment, they don't get lost (should they?).
Besides i'm not convinced that CKEDITOR.dom.element::setHtml is better
place because of 2 reasons:
1. This part of htmldataprocessor plugin doesn't use this method at all
right now.
2. Is HTML comment support really needed in CKEDITOR.dom.element::setHtml
?
What i see as a good move for next patch is moving this code to separate
function like others, eg: extendBlockForDisplay or blockNeedsExtension.
Also applying the fix only when HTML comment is present at the beginning
of data could have sense, but i'm not sure that it's really needed (would
have impact on performance).
What do you think ?
--
Ticket URL: <http://dev.fckeditor.net/ticket/3801#comment:6>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
------------------------------------------------------------------------------
_______________________________________________
FCKeditor-Trac mailing list
FCKeditor-Trac@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac
