瀏覽代碼

Moved Edge browser checking before creating icon UIElement.

Damian Konopka 7 年之前
父節點
當前提交
1d8e54f3b6
共有 1 個文件被更改,包括 6 次插入8 次删除
  1. 6 8
      packages/ckeditor5-image/src/imageupload/imageuploadprogress.js

+ 6 - 8
packages/ckeditor5-image/src/imageupload/imageuploadprogress.js

@@ -105,7 +105,8 @@ export default class ImageUploadProgress extends Plugin {
 			return;
 		}
 
-		if ( status == 'complete' && fileRepository.loaders.get( uploadId ) ) {
+		// Because in Edge there is no way to show fancy animation of completeIcon we need to skip it.
+		if ( status == 'complete' && fileRepository.loaders.get( uploadId ) && !env.isEdge ) {
 			_showCompleteIcon( viewFigure, viewWriter, editor.editing.view );
 		}
 
@@ -210,14 +211,11 @@ function _hideProgressBar( viewFigure, writer ) {
 function _showCompleteIcon( viewFigure, writer, view ) {
 	const completeIcon = new UIElement( 'div', { class: 'ck-image-upload-complete-icon' } );
 
-	// Because in Edge there is no way to show fancy animation of completeIcon we need to skip it.
-	if ( !env.isEdge ) {
-		writer.insert( ViewPosition.createAt( viewFigure, 'end' ), completeIcon );
+	writer.insert( ViewPosition.createAt( viewFigure, 'end' ), completeIcon );
 
-		setTimeout( () => {
-			view.change( writer => writer.remove( ViewRange.createOn( completeIcon ) ) );
-		}, 3000 );
-	}
+	setTimeout( () => {
+		view.change( writer => writer.remove( ViewRange.createOn( completeIcon ) ) );
+	}, 3000 );
 }
 
 // Create progress bar element using {@link module:engine/view/uielement~UIElement}.