Opened 8 years ago

Last modified 3 years ago

#6709 confirmed Bug

insertHtml: comments must also be protected

Reported by: Dinu Owned by:
Priority: Normal Milestone:
Component: General Version:
Keywords: IE8 HasPatch Cc:

Description

In IE, when setting innerHTML, any starting <!----> comments are removed. Test: insertHtml('<!--comment-->foo') Affected: dataProcessor.toHtml Fix: prefixing a dummy <foo:bar /> tag keeps comments and whitespace in place, remove when converting to data.

Change History (5)

comment:1 Changed 8 years ago by Sa'ar Zac Elias

Keywords: IE added
Status: newconfirmed

I can confirm that using the API sample.

comment:2 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: HasPatch added

comment:3 Changed 8 years ago by Garry Yao

Summary: Comments must also be protected from IEinsertHtml: comments must also be protected

comment:4 Changed 6 years ago by Jakub Ś

Other related to problems with HTML comments: #8732, #8569, #8640, #8216, #8129, #8812

Last edited 6 years ago by Jakub Ś (previous) (diff)

comment:5 Changed 3 years ago by Jakub Ś

Keywords: IE8 added; IE removed

In CKEditor 4.5.3 this issue is only reproducible IE8.

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy