Opened 10 years ago

Closed 10 years ago

#3707 closed Bug (invalid)

[webkit] editor.insertHtml not supported

Reported by: Garry Yao Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.0
Component: General Version:
Keywords: Cc:

Description

Currently we're currently rely on native document.insertHtml, but actually Webkit is not supporting this feature, check quirksmode test page.
It could be reproduced by the attached sample page.

Attachments (2)

sample_for_reproducing.html (2.1 KB) - added by Garry Yao 10 years ago.
3707.patch (4.5 KB) - added by Garry Yao 10 years ago.

Download all attachments as: .zip

Change History (4)

Changed 10 years ago by Garry Yao

Attachment: sample_for_reproducing.html added

Changed 10 years ago by Garry Yao

Attachment: 3707.patch added

comment:1 Changed 10 years ago by Garry Yao

Keywords: Review? added
Owner: set to Garry Yao
Status: newassigned

Changes contained within this patch:

  1. Bring significant change to the way we insert html string, now it's using insertElement API under the hood which bring advantages of cross-browser dtd fixing;
  2. Fixing a special case in selection.getRanges which will result in empty range when the document is initially focused;
  3. Now the undosnaps happen for both 'insertHtml' AND 'insertElement' at API level.

comment:2 Changed 10 years ago by Garry Yao

Keywords: Confirmed Review? removed
Resolution: invalid
Status: assignedclosed

It's been proved that it's only a replication of #3703, and execCommand( 'inserthtml', false, data ) works exactly as it should.So this ticket is not valid.

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