Opened 7 years ago

Closed 7 years ago

#5060 closed New Feature (fixed)

Allow registration of CKEDITOR.dialog event listeners in dialogDefinition event

Reported by: Joe Kavanagh Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.2
Component: General Version: SVN (CKEditor) - OLD
Keywords: IBM Confirmed Cc: dchojna@…

Description

Within the CKEDITOR.dialog constructor, the CKEDITOR.dialog._ object is being set after the dialogDefinition event is fired. If the dialogDefinition event is used to register an event listener for a CKEDITOR.dialog event it will get overridden, removing the CKEDITOR.dialog._.events object.

In the CKEDITOR.dialog constructor, can the block of code that sets the CKEDITOR.dialog._ object be moved above where the dialogDefinition event is fired.

Change History (3)

comment:1 Changed 7 years ago by Frederico Caldeira Knabben

Keywords: Confirmed added

comment:2 Changed 7 years ago by Garry Yao

Owner: set to Garry Yao
Status: newassigned

Well, the 'dialogDefinition' should really be used for only manipulate the definition object only.
Fixed with [4992].

comment:3 Changed 7 years ago by Garry Yao

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