Merge branch 't/10612b' into major

Author: Reinmar

CHANGES.md

@@ -5,6 +5,7 @@ CKEditor 4 Changelog
 
 Fixed Issues:
 
+* [#10612](http://dev.ckeditor.com/ticket/10612): Internet Explorer 11 compatibility issues.
 * [#10831](http://dev.ckeditor.com/ticket/10831): Image2: Merged `image2inline` and `image2block` into `image2` widget.
 
 ## CKEditor 4.3 Beta

core/env.js

@@ -28,7 +28,7 @@ if ( !CKEDITOR.env ) {
 			 *
 			 * @property {Boolean}
 			 */
-			ie: eval( '/*@cc_on!@*/false' ),
+			ie: ( agent.indexOf( 'trident/' ) > -1 ),
 			// Use eval to preserve conditional comment when compiling with Google Closure Compiler (#93).
 
 			/**
@@ -141,7 +141,7 @@ if ( !CKEDITOR.env ) {
 		 *
 		 * @property {Boolean}
 		 */
-		env.gecko = ( navigator.product == 'Gecko' && !env.webkit && !env.opera );
+		env.gecko = ( navigator.product == 'Gecko' && !env.webkit && !env.opera && !env.ie );
 
 		/**
 		 * Indicates that CKEditor is running in Chrome.

core/scriptloader.js

@@ -118,7 +118,7 @@ CKEDITOR.scriptLoader = (function() {
 						src: url } );
 
 					if ( callback ) {
-						if ( CKEDITOR.env.ie ) {
+						if ( CKEDITOR.env.ie && CKEDITOR.env.version < 11 ) {
 							// FIXME: For IE, we are not able to return false on error (like 404).
 							script.$.onreadystatechange = function() {
 								if ( script.$.readyState == 'loaded' || script.$.readyState == 'complete' ) {

core/selection.js

@@ -470,8 +470,8 @@
 					// when editor has no focus, remember this scroll
 					// position and revert it before context menu opens. (#5778)
 					if ( evt.data.$.button == 2 ) {
-						var sel = editor.document.$.selection;
-						if ( sel.type == 'None' )
+						var sel = editor.document.getSelection();
+						if ( !sel || sel.getType() == CKEDITOR.SELECTION_NONE )
 							scroll = editor.window.getScrollPosition();
 					}
 				});
@@ -552,7 +552,8 @@
 					}
 
 					// It's much simpler for IE8+, we just need to reselect the reported range.
-					if ( CKEDITOR.env.version > 7 ) {
+					// This hack does not work on IE>=11 because there's no old selection&range APIs.
+					if ( CKEDITOR.env.version > 7 && CKEDITOR.env.version < 11 ) {
 						html.on( 'mousedown', function( evt ) {
 							if ( evt.data.getTarget().is( 'html' ) ) {
 								// Limit the text selection mouse move inside of editable. (#9715)