Korbs/Squire
0
Fork 0
mirror of https://github.com/fastmail/Squire.git synced 2024-12-22 15:23:29 -05:00
Code Issues Activity

Only remove whitespace nodes if between blocks.

Browse source 
* When cleaning the tree we want to remove useless whitespace between block
  nodes, but we were being a bit too aggressive and removing all whitespace
  nodes. Now checks the context first.
This commit is contained in:
Neil Jenkins 2013-02-21 11:37:20 +11:00
parent 0bb3c142eb
commit 7162777a0f
2 changed files with 9 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
build/squire.js
View file

File diff suppressed because one or more lines are too long

15
source/Editor.js
View file

@ -1148,13 +1148,14 @@
if ( childLength ) { if ( childLength ) {
cleanTree( child, allowStyles ); cleanTree( child, allowStyles );
} }
} else { } else if ( nodeType !== TEXT_NODE || (
if ( ( nodeType !== TEXT_NODE ) || !( /\S/.test( child.data ) ) &&
!( /\S/.test( child.data ) ) ) { !( i > 0 && children[ i - 1 ].isInline() ) &&
node.removeChild( child ); !( i + 1 < l && children[ i + 1 ].isInline() )
i -= 1; ) ) {
l -= 1; node.removeChild( child );
} i -= 1;
l -= 1;
} }
} }
return node; return node;