Bladeren bron

Fired document#layoutChanged instead of rendering the view.

Oskar Wróbel 7 jaren geleden
bovenliggende
commit
bfca9a28ec

+ 1 - 1
packages/ckeditor5-image/src/image/imageloadobserver.js

@@ -77,7 +77,7 @@ export default class ImageLoadObserver extends DomEventObserver {
 	 * @inheritDoc
 	 */
 	onDomEvent( domEvent ) {
-		this.view.render();
+		this.document.fire( 'layoutChanged' );
 		this.fire( 'imageLoaded', domEvent );
 	}
 

+ 4 - 4
packages/ckeditor5-image/tests/image/imageloadobserver.js

@@ -81,16 +81,16 @@ describe( 'ClickObserver', () => {
 		sinon.assert.neverCalledWith( mapSpy, viewDiv );
 	} );
 
-	it( 'should call `view#render` along with `imageLoaded` event', () => {
-		const renderSpy = sinon.spy();
+	it( 'should call `layoutChanged` along with `imageLoaded` event', () => {
+		const layoutChangedSpy = sinon.spy();
 		const imageLoadedSpy = sinon.spy();
 
-		view.on( 'render', renderSpy );
+		view.document.on( 'layoutChanged', layoutChangedSpy );
 		view.document.on( 'imageLoaded', imageLoadedSpy );
 
 		observer.onDomEvent( {} );
 
-		sinon.assert.calledOnce( renderSpy );
+		sinon.assert.calledOnce( layoutChangedSpy );
 		sinon.assert.calledOnce( imageLoadedSpy );
 	} );
 } );