Opened 8 years ago

Closed 8 years ago

#3160 closed Bug (fixed)

[FF] Ajax sample editor document problem

Reported by: Garry Yao Owned by: Frederico Caldeira Knabben
Priority: Normal Milestone: CKEditor 3.0
Component: General Version:
Keywords: Confirmed Firefox Review+ Cc:

Description

Reproducing Procedures

  1. Open the ajax.html example page;
  2. Click Create Editor button to have a new instance;
  3. Insert some content and click Remove Editor to destroy the instance;
  4. Click Create Editor button again to have another new instance;
  • Actual Result : The content is lost within the new instance and the document become not editable.

Attachments (2)

3160.patch (467 bytes) - added by Garry Yao 8 years ago.
3160_2.patch (1.2 KB) - added by Frederico Caldeira Knabben 8 years ago.

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by Garry Yao

Attachment: 3160.patch added

comment:1 Changed 8 years ago by Garry Yao

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

Fixing the bug by avoiding cross-frame scripting security error from generated when invoke editor.setData().

comment:2 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed
Owner: changed from Garry Yao to Frederico Caldeira Knabben
Status: assignednew

While this patch solves the problem for this specific case, I think something more generic would be better. The fact is that, as soon as "mode" gets fired, the code should be able to interact with the mode contents without setTimeout hacks. Currently this is not doable in the "wysiwyg" mode, the browser needs time to render the document first.

I've already coded it in a new patch that's based on Garry's one. I'm taking over the ticket for it.

Changed 8 years ago by Frederico Caldeira Knabben

Attachment: 3160_2.patch added

comment:3 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review? added; Review- removed

comment:4 Changed 8 years ago by Garry Yao

Keywords: Review+ added; Review? removed
Version: SVN (FCKeditor)

comment:5 Changed 8 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: newclosed

Fixed with [3254].

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