Szymon Kupś před 8 roky
rodič
revize
37b52b4cc7

+ 4 - 1
packages/ckeditor5-autoformat/src/inlineautoformatengine.js

@@ -198,7 +198,10 @@ export default class InlineAutoformatEngine {
 				// Remove delimiters.
 				// Remove delimiters.
 				for ( const range of rangesToRemove ) {
 				for ( const range of rangesToRemove ) {
 					fixBatch.remove( range );
 					fixBatch.remove( range );
-					range.detach(); // Prevents memory leaks. #39
+
+					// Prevents memory leaks.
+					// https://github.com/ckeditor/ckeditor5-autoformat/issues/39
+					range.detach();
 				}
 				}
 			} );
 			} );
 		} );
 		} );

+ 1 - 1
packages/ckeditor5-autoformat/tests/inlineautoformatengine.js

@@ -150,7 +150,7 @@ describe( 'InlineAutoformatEngine', () => {
 				doc.batch().insert( doc.selection.getFirstPosition(), '*' );
 				doc.batch().insert( doc.selection.getFirstPosition(), '*' );
 			} );
 			} );
 
 
-			// There should be two ranges removed
+			// There should be two ranges removed.
 			expect( detachSpies ).to.have.length( 2 );
 			expect( detachSpies ).to.have.length( 2 );
 
 
 			for ( const spy of detachSpies ) {
 			for ( const spy of detachSpies ) {