|
|
@@ -1141,6 +1141,70 @@ describe( 'DocumentSelection', () => {
|
|
|
} );
|
|
|
} );
|
|
|
|
|
|
+ // https://github.com/ckeditor/ckeditor5-engine/issues/1673
|
|
|
+ describe( 'refreshing selection data', () => {
|
|
|
+ it( 'should be up to date when post fixers are called', done => {
|
|
|
+ model.schema.extend( '$text', { allowAttributes: 'foo' } );
|
|
|
+
|
|
|
+ const p = doc.getRoot().getChild( 0 );
|
|
|
+
|
|
|
+ doc.registerPostFixer( () => {
|
|
|
+ expect( model.document.selection.getAttribute( 'foo' ) ).to.equal( 'bar' );
|
|
|
+ expect( Array.from( model.document.selection.markers, m => m.name ) ).to.deep.equal( [ 'marker' ] );
|
|
|
+ done();
|
|
|
+ } );
|
|
|
+
|
|
|
+ model.change( writer => {
|
|
|
+ writer.insertText( 'abcdef', { foo: 'bar' }, p );
|
|
|
+
|
|
|
+ writer.addMarker( 'marker', {
|
|
|
+ range: writer.createRange(
|
|
|
+ writer.createPositionFromPath( p, [ 1 ] ),
|
|
|
+ writer.createPositionFromPath( p, [ 5 ] )
|
|
|
+ ),
|
|
|
+ usingOperation: false
|
|
|
+ } );
|
|
|
+
|
|
|
+ writer.setSelection( writer.createPositionFromPath( p, [ 3 ] ) );
|
|
|
+ } );
|
|
|
+ } );
|
|
|
+
|
|
|
+ it( 'should be up to date when post fixers have changed the data', () => {
|
|
|
+ model.schema.extend( '$text', { allowAttributes: 'foo' } );
|
|
|
+
|
|
|
+ const p = doc.getRoot().getChild( 0 );
|
|
|
+
|
|
|
+ doc.registerPostFixer( writer => {
|
|
|
+ writer.setAttribute( 'foo', 'biz', p.getChild( 0 ) );
|
|
|
+ writer.removeMarker( 'marker-1' );
|
|
|
+ writer.addMarker( 'marker-2', {
|
|
|
+ range: writer.createRange(
|
|
|
+ writer.createPositionFromPath( p, [ 1 ] ),
|
|
|
+ writer.createPositionFromPath( p, [ 5 ] )
|
|
|
+ ),
|
|
|
+ usingOperation: false
|
|
|
+ } );
|
|
|
+ } );
|
|
|
+
|
|
|
+ model.change( writer => {
|
|
|
+ writer.insertText( 'abcdef', { foo: 'bar' }, p );
|
|
|
+
|
|
|
+ writer.addMarker( 'marker-1', {
|
|
|
+ range: writer.createRange(
|
|
|
+ writer.createPositionFromPath( p, [ 1 ] ),
|
|
|
+ writer.createPositionFromPath( p, [ 5 ] )
|
|
|
+ ),
|
|
|
+ usingOperation: false
|
|
|
+ } );
|
|
|
+
|
|
|
+ writer.setSelection( writer.createPositionFromPath( p, [ 3 ] ) );
|
|
|
+ } );
|
|
|
+
|
|
|
+ expect( model.document.selection.getAttribute( 'foo' ) ).to.equal( 'biz' );
|
|
|
+ expect( Array.from( model.document.selection.markers, m => m.name ) ).to.deep.equal( [ 'marker-2' ] );
|
|
|
+ } );
|
|
|
+ } );
|
|
|
+
|
|
|
// DocumentSelection uses LiveRanges so here are only simple test to see if integration is
|
|
|
// working well, without getting into complicated corner cases.
|
|
|
describe( 'after applying an operation should get updated and fire events', () => {
|