Explorar o código

Small refactoring, docs fixes.

Szymon Kupś %!s(int64=8) %!d(string=hai) anos
pai
achega
5a49ad9dbe
Modificáronse 1 ficheiros con 6 adicións e 6 borrados
  1. 6 6
      packages/ckeditor5-engine/src/view/renderer.js

+ 6 - 6
packages/ckeditor5-engine/src/view/renderer.js

@@ -381,9 +381,9 @@ export default class Renderer {
 			return false;
 			return false;
 		}
 		}
 
 
-		// Block adding inline filler inside elements with contenteditable=false.
+		// Prevent adding inline filler inside elements with contenteditable=false.
 		// https://github.com/ckeditor/ckeditor5-engine/issues/1170
 		// https://github.com/ckeditor/ckeditor5-engine/issues/1170
-		if ( _isNotEditable( selectionParent ) ) {
+		if ( !_isEditable( selectionParent ) ) {
 			return false;
 			return false;
 		}
 		}
 
 
@@ -715,17 +715,17 @@ export default class Renderer {
 
 
 mix( Renderer, ObservableMixin );
 mix( Renderer, ObservableMixin );
 
 
-// Checks if contents inside provided element are not editable.
+// Checks if provided element is editable.
 //
 //
 // @private
 // @private
 // @param {module:engine/view/element~Element} element
 // @param {module:engine/view/element~Element} element
 // @returns {Boolean}
 // @returns {Boolean}
-function _isNotEditable( element ) {
+function _isEditable( element ) {
 	if ( element.getAttribute( 'contenteditable' ) == 'false' ) {
 	if ( element.getAttribute( 'contenteditable' ) == 'false' ) {
-		return true;
+		return false;
 	}
 	}
 
 
 	const parent = element.findAncestor( element => element.hasAttribute( 'contenteditable' ) );
 	const parent = element.findAncestor( element => element.hasAttribute( 'contenteditable' ) );
 
 
-	return parent && parent.getAttribute( 'contenteditable' ) == 'false';
+	return !parent || parent.getAttribute( 'contenteditable' ) == 'true';
 }
 }