Opened 17 years ago

Closed 17 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 17 years ago.
3707.patch​ (4.5 KB) - added by Garry Yao 17 years ago.

Download all attachments as: .zip

Change History (4)

Changed 17 years ago by Garry Yao

Attachment: sample_for_reproducing.html​ added

Changed 17 years ago by Garry Yao

Attachment: 3707.patch​ added

comment:1 Changed 17 years ago by Garry Yao

Keywords: Review? added
Owner: set to Garry Yao
Status: new → assigned

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 17 years ago by Garry Yao

Keywords: Confirmed Review? removed
Resolution: → invalid
Status: assigned → closed

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 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy