Other: Remove the complicated innerText.length calculations, required…

… due to old Edge quirks. ckeditor/ckeditor5#6202. Agreed at ckeditor/ckeditor5-utils#333 (comment)
ckeditor · Apr 20, 2020 · 572aaa9 · 572aaa9
1 parent f38afc0
commit 572aaa9
Showing 1 changed file with 1 addition and 6 deletions.
diff --git a/src/filters/space.js b/src/filters/space.js
@@ -39,12 +39,7 @@ export function normalizeSpacing( htmlString ) {
  */
 export function normalizeSpacerunSpans( htmlDocument ) {
 	htmlDocument.querySelectorAll( 'span[style*=spacerun]' ).forEach( el => {
-		// Use `el.childNodes[ 0 ].data.length` instead of `el.innerText.length`. For `el.innerText.length` which
-		// contains spaces mixed with `&nbsp;` Edge browser returns incorrect length.
-		const innerTextLength = ( el.childNodes &&
-			el.childNodes[ 0 ] &&
-			el.childNodes[ 0 ].data &&
-			el.childNodes[ 0 ].data.length ) || 0;
+		const innerTextLength = el.innerText.length || 0;
 
 		el.innerHTML = Array( innerTextLength + 1 ).join( '\u00A0 ' ).substr( 0, innerTextLength );
 	} );