Browse Source

Merge branch 'master' into t/1439

Aleksander Nowodzinski 7 years ago
parent
commit
8f95368e1e

+ 1 - 1
packages/ckeditor5-engine/src/model/writer.js

@@ -204,7 +204,7 @@ export default class Writer {
 					markerRange.end._getCombined( rangeRootPosition, position )
 				);
 
-				this.addMarker( markerName, { range, usingOperation: true } );
+				this.addMarker( markerName, { range, usingOperation: true, affectsData: true } );
 			}
 		}
 	}

+ 4 - 1
packages/ckeditor5-engine/tests/model/writer.js

@@ -310,10 +310,13 @@ describe( 'Writer', () => {
 
 			expect( Array.from( model.markers ).length ).to.equal( 1 );
 
-			const range = model.markers.get( 'marker' ).getRange();
+			const modelMarker = model.markers.get( 'marker' );
+			const range = modelMarker.getRange();
 			expect( range.root ).to.equal( root );
 			expect( range.start.path ).to.deep.equal( [ 2, 1 ] );
 			expect( range.end.path ).to.deep.equal( [ 2, 5 ] );
+			expect( modelMarker.managedUsingOperations ).to.be.true;
+			expect( modelMarker.affectsData ).to.be.true;
 		} );
 
 		it( 'should throw when trying to use detached writer', () => {