Merge branch 't/11727' into major

CHANGES.md

@@ -31,6 +31,10 @@ New Features:
   If defined, the editor produces classes instead of inline styles for aligned images.
   * Default caption of the image can be translated (customized) with `editor.lang.image2.captionPlaceholder`.
 
+Fixed Issues:
+
+* [#11727](http://dev.ckeditor.com/ticket/11727): Fixed: Editor tries to select non-editable image which was clicked.
+
 ## CKEditor 4.3.5
 
 Fixed Issues:

core/editable.js

@@ -753,7 +753,10 @@
 				if ( !CKEDITOR.env.ie ) {
 					this.attachListener( this, 'mousedown', function( ev ) {
 						var control = ev.data.getTarget();
-						if ( control.is( 'img', 'hr', 'input', 'textarea', 'select' ) ) {
+						// #11727. Note: htmlDP assures that input/textarea/select have contenteditable=false
+						// attributes. However, they also have data-cke-editable attribute, so isReadOnly() returns false,
+						// and therefore those elements are correctly selected by this code.
+						if ( control.is( 'img', 'hr', 'input', 'textarea', 'select' ) && !control.isReadOnly() ) {
 							editor.getSelection().selectElement( control );
 
 							// Prevent focus from stealing from the editable. (#9515)