Преглед изворни кода

Simplified checking if range stick out of given range.

Oskar Wróbel пре 9 година
родитељ
комит
6f77ae95a5
1 измењених фајлова са 7 додато и 9 уклоњено
  1. 7 9
      packages/ckeditor5-engine/src/view/writer.js

+ 7 - 9
packages/ckeditor5-engine/src/view/writer.js

@@ -426,7 +426,7 @@ export function clear( range, element ) {
 	validateRangeContainer( range );
 
 	// Create walker on given range.
-	// We walk backward because when we remove element during walk it modify range end position.
+	// We walk backward because when we remove element during walk it modifies range end position.
 	const walker = range.getWalker( {
 		direction: 'backward',
 		ignoreElementEnd: true
@@ -457,14 +457,12 @@ export function clear( range, element ) {
 		// If we have found element to remove.
 		if ( rangeToRemove ) {
 			// We need to check if element range stick out of the given range and truncate if it is.
-			if ( !range.containsRange( rangeToRemove ) ) {
-				if ( rangeToRemove.end.isAfter( range.end ) ) {
-					rangeToRemove.end = range.end;
-				}
-
-				if ( rangeToRemove.start.isBefore( range.start ) ) {
-					rangeToRemove.start = range.start;
-				}
+			if ( rangeToRemove.end.isAfter( range.end ) ) {
+				rangeToRemove.end = range.end;
+			}
+
+			if ( rangeToRemove.start.isBefore( range.start ) ) {
+				rangeToRemove.start = range.start;
 			}
 
 			// At the end we remove range with found element.