Korbs/Squire
0
Fork 0
mirror of https://github.com/fastmail/Squire.git synced 2025-01-03 05:00:13 -05:00
Code Issues Activity

Fix first line of plain text paste not HTML escaped

Browse source
This commit is contained in:
Neil Jenkins 2020-03-10 18:46:40 +11:00
parent f6d6ac8ca0
commit 2d307ba54a
3 changed files with 15 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
build/squire-raw.js
View file

@ -4579,13 +4579,16 @@ proto.insertPlainText = function ( plainText, isPaste ) {
}
openBlock += '>';
// We don't wrap the first line in the block, so if it gets inserted into
// a blank line it keeps that line's formatting.
for ( i = 1, l = lines.length; i < l; i += 1 ) {
for ( i = 0, l = lines.length; i < l; i += 1 ) {
line = lines[i];
line = escapeHTML( line ).replace( / (?= )/g, '&nbsp;' );
// We don't wrap the first line in the block, so if it gets inserted
// into a blank line it keeps that line's formatting.
// Wrap each line in <div></div>
lines[i] = openBlock + ( line || '<BR>' ) + closeBlock;
if ( i ) {
line = openBlock + ( line || '<BR>' ) + closeBlock;
}
lines[i] = line;
}
return this.insertHTML( lines.join( '' ), isPaste );
};

2
build/squire.js
View file

File diff suppressed because one or more lines are too long

11
source/Editor.js
View file

@ -2020,13 +2020,16 @@ proto.insertPlainText = function ( plainText, isPaste ) {
}
openBlock += '>';
// We don't wrap the first line in the block, so if it gets inserted into
// a blank line it keeps that line's formatting.
for ( i = 1, l = lines.length; i < l; i += 1 ) {
for ( i = 0, l = lines.length; i < l; i += 1 ) {
line = lines[i];
line = escapeHTML( line ).replace( / (?= )/g, '&nbsp;' );
// We don't wrap the first line in the block, so if it gets inserted
// into a blank line it keeps that line's formatting.
// Wrap each line in <div></div>
lines[i] = openBlock + ( line || '<BR>' ) + closeBlock;
if ( i ) {
line = openBlock + ( line || '<BR>' ) + closeBlock;
}
lines[i] = line;
}
return this.insertHTML( lines.join( '' ), isPaste );
};