Ticket #7932: 7932.patch

_source/plugins/clipboard/plugin.js

@@ -226 +226 @@
                 // Wait a while and grab the pasted contents
                 window.setTimeout( function()
                 {
-                        mode == 'text' && CKEDITOR.env.gecko && editor.focusGrabber.focus();
+                        // Gecko need to move focus back from the textarea
+                        // to blink the cursor. (#5684)
+                        if ( mode == 'text' && CKEDITOR.env.gecko )
+                                editor.document.getBody().focus();
+
                         pastebin.remove();
                         editor.removeListener( 'selectionChange', cancel );
 

_source/plugins/wysiwygarea/plugin.js

@@ -703 +703 @@
                                                         } );
                                                 }
 
-                                                // IE standard compliant in editing frame doesn't focus the editor when
-                                                // clicking outside actual content, manually apply the focus. (#1659)
-                                                if ( editable &&
-                                                                CKEDITOR.env.ie && domDocument.$.compatMode == 'CSS1Compat'
-                                                                || CKEDITOR.env.gecko
-                                                                || CKEDITOR.env.opera )
-                                                {
-                                                        var htmlElement = domDocument.getDocumentElement();
-                                                        htmlElement.on( 'mousedown', function( evt )
-                                                        {
-                                                                // Setting focus directly on editor doesn't work, we
-                                                                // have to use here a temporary element to 'redirect'
-                                                                // the focus.
-                                                                if ( evt.data.getTarget().equals( htmlElement ) )
-                                                                {
-                                                                        if ( CKEDITOR.env.gecko && CKEDITOR.env.version >= 10900 )
-                                                                                blinkCursor();
-                                                                        focusGrabber.focus();
-                                                                }
-                                                        } );
-                                                }
-
                                                 var focusTarget = CKEDITOR.env.ie ? iframe : domWindow;
                                                 focusTarget.on( 'blur', function()
                                                         {
@@ -1205 +1183 @@
                                         });
                         }
 
-                        // Create an invisible element to grab focus.
-                        if ( CKEDITOR.env.gecko || CKEDITOR.env.ie || CKEDITOR.env.opera )
-                        {
-                                var focusGrabber;
-                                editor.on( 'uiReady', function()
-                                {
-                                        focusGrabber = editor.container.append( CKEDITOR.dom.element.createFromHtml(
-                                                // Use 'span' instead of anything else to fly under the screen-reader radar. (#5049)
-                                                '<span tabindex="-1" style="position:absolute;" role="presentation"></span>' ) );
-
-                                        focusGrabber.on( 'focus', function()
-                                                {
-                                                        editor.focus();
-                                                } );
-
-                                        editor.focusGrabber = focusGrabber;
-                                } );
-                                editor.on( 'destroy', function()
-                                {
-                                        CKEDITOR.tools.removeFunction( contentDomReadyHandler );
-                                        focusGrabber.clearCustomData();
-                                        delete editor.focusGrabber;
-                                } );
-                        }
-
                         // Disable form elements editing mode provided by some browers. (#5746)
                         editor.on( 'insertElement', function ( evt )
                         {

_source/plugins/selection/plugin.js

@@ -399 +399 @@
                                                                 saveSelection();
                                                         });
 
+                                                // When content doc is in standards mode, IE doesn't focus the editor when
+                                                // clicking on outside body (on html element) content, manually apply the
+                                                // selection helps. (#1659, #7932)
+                                                html.on( 'mouseup', function()
+                                                {
+                                                        var sel = CKEDITOR.document.$.selection,
+                                                                range = sel.createRange();
+                                                        // The selection range is reported on host, but actually it should applies to the content doc.
+                                                        if ( sel.type != 'None' && range.parentElement().ownerDocument == doc.$ )
+                                                                range.select();
+                                                } );
 
                                                 // IE is the only to provide the "selectionchange"
                                                 // event.