Adjusted special 'spacerun span' elements normalization function for …

…Edge browser.
ckeditor · Oct 26, 2018 · 316f893 · 316f893
1 parent eca95b1
commit 316f893
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/filters/utils.js b/src/filters/utils.js
@@ -109,8 +109,10 @@ function normalizeEndTagsPrecedingSpace( htmlString ) {
 // @param {Document} htmlDocument Native `Document` object in which spacing should be normalized.
 function normalizeSpacerunSpans( htmlDocument ) {
 	htmlDocument.querySelectorAll( 'span[style*=spacerun]' ).forEach( el => {
-		const spacesReplacemanet = Array( el.innerText.length + 1 ).join( '\u00A0 ' ).substr( 0, el.innerText.length );
+		// 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[ 0 ].data.length;
 
-		el.innerHTML = spacesReplacemanet;
+		el.innerHTML = Array( innerTextLength + 1 ).join( '\u00A0 ' ).substr( 0, innerTextLength );
 	} );
 }