Ticket #4812: 4812_2.patch

_source/plugins/dialog/plugin.js

@@ -641 +641 @@
                                 addCover( this._.editor );
 
                                 element.on( 'keydown', accessKeyDownHandler );
-                                element.on( 'keyup', accessKeyUpHandler );
+                                element.on( CKEDITOR.env.opera || ( CKEDITOR.env.gecko && CKEDITOR.env.mac ) ?
+                                            'keypress' : 'keyup' , accessKeyUpHandler );
 
                                 // Prevent some keys from bubbling up. (#4269)
                                 for ( var event in { keyup :1, keydown :1, keypress :1 } )
                                         element.on( event, preventKeyBubbling );
-                        }
+        }
                         else
                         {
                                 this._.parentDialog = CKEDITOR.dialog._.currentTop;
@@ -771 +772 @@
 
                                 // Remove access key handlers.
                                 element.removeListener( 'keydown', accessKeyDownHandler );
-                                element.removeListener( 'keyup', accessKeyUpHandler );
-                                element.removeListener( 'keypress', accessKeyUpHandler );
+                                element.removeListener( CKEDITOR.env.opera || ( CKEDITOR.env.gecko && CKEDITOR.env.mac ) ?
+                                            'keypress' : 'keyup', accessKeyUpHandler );
 
                                 // Remove bubbling-prevention handler. (#4269)
                                 for ( var event in { keyup :1, keydown :1, keypress :1 } )
@@ -1832 +1833 @@
                         return;
 
                 keyProcessor = keyProcessor[keyProcessor.length - 1];
-                keyProcessor.keyup && keyProcessor.keyup.call( keyProcessor.uiElement, keyProcessor.dialog, keyProcessor.key );
-                evt.data.preventDefault();
+                if( keyProcessor.keyup )
+                {
+                        keyProcessor.keyup.call( keyProcessor.uiElement, keyProcessor.dialog, keyProcessor.key );
+                        evt.data.preventDefault( );
+                }
         };
 
         var registerAccessKey = function( uiElement, dialog, key, downFunc, upFunc )