8
0
Просмотр исходного кода

Merge branch i/6202-remove-env.isEdge to master.

Internal: Removed mentions on Edge. See ckeditor/ckeditor5#6202.
Piotrek Koszuliński 5 лет назад
Родитель
Сommit
446170addd

+ 1 - 2
packages/ckeditor5-engine/src/view/domconverter.js

@@ -169,8 +169,7 @@ export default class DomConverter {
 			this._domToViewMapping.delete( domElement );
 			this._viewToDomMapping.delete( viewElement );
 
-			// Use Array.from because of MS Edge (#923).
-			for ( const child of Array.from( domElement.childNodes ) ) {
+			for ( const child of domElement.childNodes ) {
 				this.unbindDomElement( child );
 			}
 		}

+ 1 - 1
packages/ckeditor5-engine/tests/view/renderer.js

@@ -2056,7 +2056,7 @@ describe( 'Renderer', () => {
 
 			// Use a forgiving way of checking what the selection contains
 			// because Safari normalizes the selection ranges so precise checking is troublesome.
-			// Also, Edge returns a normal space instead of nbsp so we need to use even more alternatives.
+			// Also, Firefox returns a normal space instead of nbsp so we need to use even more alternatives.
 			function assertDomSelectionContents( domSelection, expectedContainer, expectedText ) {
 				const domSelectionContainer = domSelection.getRangeAt( 0 ).commonAncestorContainer;