Opened 6 years ago

Closed 6 years ago

#3810 closed Bug (fixed)

IE: Template throws error

Reported by: fredck Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.0
Component: General Version: SVN (CKEditor) - OLD
Keywords: Confirmed IE Review? WorksForMe Cc:


With IE, a js error is thrown when using the Template button, after selecting one of the templates.

Attachments (1)

3810.patch (1.9 KB) - added by garry.yao 6 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 6 years ago by garry.yao

  • Owner set to garry.yao
  • Status changed from new to assigned

Problem comes from 'CKEDITOR.editor::setData'.

Changed 6 years ago by garry.yao

comment:2 Changed 6 years ago by garry.yao

  • Keywords Review? added

Again, coz CKEDITOR.editor::setData is sync, unlock selection ( invoked in dialog.hide ) for IE will result in error, the only event we can relay on now is contentDom, so I make the changes to template plugin.
Then, I found the editor is still not gaining focus within the event handler, and that's because contentDom is fired too early to gain document focus.( Focus is locked by 'isLoadingData')Ok, so let's make the event happened later, result in such a ugly fix.

comment:3 Changed 6 years ago by arczi

  • Keywords WorksForMe added

This bug seems to be fixed with [3794]

comment:4 Changed 6 years ago by fredck

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

Yes, it's not anymore an issue with the current trunk.

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