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

Fix range error in addFormat if start == end container.

Browse source
This commit is contained in:
Neil Jenkins 2014-10-03 16:38:20 +07:00
parent b944eb3b3b
commit 73c4146e27
3 changed files with 9 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
build/squire-raw.js
View file

@ -1709,6 +1709,10 @@ proto._addFormat = function ( tag, attributes, range ) {
} }
if ( textNode === startContainer && startOffset ) { if ( textNode === startContainer && startOffset ) {
textNode = textNode.splitText( startOffset ); textNode = textNode.splitText( startOffset );
if ( endContainer === startContainer ) {
endContainer = textNode;
endOffset -= startOffset;
}
startContainer = textNode; startContainer = textNode;
startOffset = 0; startOffset = 0;
} }

2
build/squire.js
View file

File diff suppressed because one or more lines are too long

4
source/Editor.js
View file

@ -624,6 +624,10 @@ proto._addFormat = function ( tag, attributes, range ) {
} }
if ( textNode === startContainer && startOffset ) { if ( textNode === startContainer && startOffset ) {
textNode = textNode.splitText( startOffset ); textNode = textNode.splitText( startOffset );
if ( endContainer === startContainer ) {
endContainer = textNode;
endOffset -= startOffset;
}
startContainer = textNode; startContainer = textNode;
startOffset = 0; startOffset = 0;
} }