Ticket #3219: 3219_2.patch

_source/plugins/dialog/plugin.js

@@ -517 +517 @@
                         else
                                 return;
 
+                        CKEDITOR.fire( 'beforeDialogOpen', definition );
+
                         // First, set the dialog to an appropriate size.
                         this.resize( definition.minWidth, definition.minHeight );
 

_source/plugins/floatpanel/plugin.js

@@ -64 +64 @@
                                 children : [],
                                 dir : editor.lang.dir
                         };
+                        
+                        CKEDITOR.on( "beforeDialogOpen" , function() 
+                                { 
+                                        this.hide(); 
+                                }, this ); 
                 },
 
                 proto :
@@ -124 +129 @@
                                                 display : ''
                                         });
 
+                                // Non IE prefer the event into a window object.
+                                var focusElement = CKEDITOR.env.ie ? iframe : new CKEDITOR.dom.window( iframe.$.contentWindow );
+
                                 // Configure the IFrame blur event. Do that only once.
                                 if ( !this._.blurSet )
                                 {
-                                        // Non IE prefer the event into a window object.
-                                        var focused = CKEDITOR.env.ie ? iframe : new CKEDITOR.dom.window( iframe.$.contentWindow );
+                                        focusElement.on( 'blur', function()
+                                                {
+                                                        if ( !this._.panel.allowBlur )
+                                                                return;
 
-                                        focused.on( 'blur', function()
-                                                {
                                                         if ( !this._.activeChild )
                                                                 this.hide();
                                                 },
                                                 this );
 
-                                        focused.on( 'focus', function()
+                                        focusElement.on( 'focus', function()
                                                 {
                                                         this._.focused = true;
                                                         this.hideChild();
+                                                        this.allowBlur( true );
                                                 },
                                                 this );
 
@@ -183 +192 @@
                                                 // Set the IFrame focus, so the blur event gets fired.
                                                 setTimeout( function()
                                                         {
-                                                                iframe.$.contentWindow.focus();
+                                                                focusElement.focus();
                                                         }, 0);
                                         }, 0);
-
                                 if ( this.onShow )
                                         this.onShow.call( this );
                         },
 
+                        allowBlur : function( allow )
+                        {
+                                this._.panel.allowBlur = allow;
+                        },
+
                         hide : function()
                         {
                                 if ( !this.onHide || this.onHide.call( this ) !== true )

_source/plugins/panel/plugin.js

@@ -260 +260 @@
 
                 hide : function()
                 {
-                        this.element.setStyle( 'display', 'none' );
+                        if ( !this.onHide || this.onHide.call( this )  !== true )
+                                this.element.setStyle( 'display', 'none' );
                 },
 
                 onKeyDown : function( keystroke )

_source/plugins/panelbutton/plugin.js

@@ -85 +85 @@
 
                                         this.createPanel( editor );
 
+                                        var panel = _.panel;
+                                        panel.allowBlur( false );
+
                                         if ( _.on )
                                         {
-                                                _.panel.hide();
+                                                panel.hide();
                                                 return;
                                         }
+                                        panel.showBlock( this._.id, new CKEDITOR.dom.element( $element ), 4 );
 
-                                        _.panel.showBlock( this._.id, new CKEDITOR.dom.element( $element ), 4 );
+                                        if ( !CKEDITOR.env.ie )
+                                        {
+                                                setTimeout( function()
+                                                        {
+                                                                if (CKEDITOR.env.gecko )
+                                                                        panel._.iframe.$.focus();                                       // Force blur.
+                                                                panel.allowBlur( true );
+                                                        }, 0 );
+                                        }
+
                                 },
                                 this );
                         var keyDownFn = CKEDITOR.tools.addFunction( function( ev, element ){
@@ -210 +223 @@
                                         me.document.getById( _.id ).focus();
                                 };
 
-
                         if ( this.onBlock )
                                 this.onBlock( panel, _.id );
+
+                        // On block hide.
+                        panel.getBlock( _.id ).onHide = function()
+                                {
+                                        _.on = 0;
+                                        me.setState( CKEDITOR.TRISTATE_OFF );
+                                }
                 },
 
                 setState : CKEDITOR.ui.button.prototype.setState

_source/plugins/richcombo/plugin.js

@@ -93 +93 @@
 
                                         this.createPanel( editor );
 
+                                        var panel = _.panel;
+                                        panel.allowBlur( false );
+
                                         if ( _.on )
                                         {
-                                                _.panel.hide();
+                                                panel.hide();
                                                 return;
                                         }
 
@@ -111 +114 @@
                                         else
                                                 _.list.unmarkAll();
 
-                                        _.panel.showBlock( this.id, new CKEDITOR.dom.element( $element ).getFirst(), 4 );
+                                        panel.showBlock( this.id, new CKEDITOR.dom.element( $element ).getFirst(), 4 );
+
+                                        if ( !CKEDITOR.env.ie )
+                                        {
+                                                setTimeout( function()
+                                                        {
+                                                                if ( CKEDITOR.env.gecko )
+                                                                        panel._.iframe.$.focus();                                       // Force blur.
+                                                                panel.allowBlur( true );
+                                                        }, 0);
+                                        }
                                 },
                                 this );
 
@@ -264 +277 @@
                         this._.panel = panel;
                         this._.list = list;
 
+                        // On block hide.
+                        panel.getBlock( this.id ).onHide = function()
+                                {
+                                        me._.on = 0;
+                                        me.setState( CKEDITOR.TRISTATE_OFF );
+                                }
+
                         if ( this.init )
                                 this.init();
                 },