How to add a selectPage listener?

I want to add a listener of selectPage event to my CKEditor plugin, but so far I'm unable to get it to work. The plugin works fine but my listener is not working. The alert below never gets called.

The code:

    var dialog = function(editor) {

        editor.on('selectPage', function(e) {
            alert("Page changed");
        });

         // rest of the plugin code
    }

Is there something I'm doing wrong?

Thu, 09/15/2011 - 17:03

alfonsoml

Joined: 31/12/2006

Posts: 3737

Re: How to add a selectPage listener?

selectPage

is an event of a dialog instance, it will not fire on the editor itself.

Thu, 09/15/2011 - 17:38

mark85

Joined: 15/09/2011

Posts: 5

Re: How to add a selectPage listener?

Thank you for the reply, could you tell me what's the correct place to implement this?

I've tried it the following ways, but it won't work

        var dialog = function(editor) {
        this.on('selectPage', function(e) {
            alert("Page changed");
        });

         // rest of the plugin code
    }

and

CKEDITOR.dialog.add('plugin', function(editor) {
   var d = dialog(editor); // call to the dialog function
   d.on('selectPage', function(e) {
      alert("Page changed");
   });
   return d;
});

Thu, 09/15/2011 - 18:15

alfonsoml

Joined: 31/12/2006

Posts: 3737

Re: How to add a selectPage listener?

The object that you are trying to use there is the dialog definition, it's not the real dialog object itself. Yes weird and troublesome, but that's the way it is.

You can get the real object for example by using an onLoad function:

      return {
         title : editor.lang.googlemaps.title,
         minWidth : 500,
         minHeight : 460,
         onLoad : function()
         {
            theDialog = this;

            // Act on tab switching
            theDialog.on('selectPage', function (e)
               {
                  if (CKEDITOR.env.ie7Compat)
                     fixIE7display();

The trick there is that when that onLoad is executed, it will be done on the real dialog, so "this" will work.

Fri, 09/16/2011 - 10:44

mark85

Joined: 15/09/2011

Posts: 5

Re: How to add a selectPage listener?

Thanks a lot, works perfectly now!