Ticket #3160 (closed Bug: fixed)

Opened 5 years ago

Last modified 5 years ago

[FF] Ajax sample editor document problem

Reported by: garry.yao Owned by: fredck
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

3160.patch (467 bytes) - added by garry.yao 5 years ago.
3160_2.patch (1.2 KB) - added by fredck 5 years ago.

Change History

Changed 5 years ago by garry.yao

comment:1 Changed 5 years ago by garry.yao

  • Status changed from new to assigned
  • Keywords Review? added
  • Owner set to garry.yao

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

comment:2 Changed 5 years ago by fredck

  • Owner changed from garry.yao to fredck
  • Keywords Review- added; Review? removed
  • Status changed from assigned to new

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 5 years ago by fredck

comment:3 Changed 5 years ago by fredck

  • Keywords Review? added; Review- removed

comment:4 Changed 5 years ago by garry.yao

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

comment:5 Changed 5 years ago by fredck

  • Status changed from new to closed
  • Resolution set to fixed

Fixed with [3254].

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