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

Handle empty nodes in moveRangeBoundariesDownTree

Browse source 
Empty text nodes next to another text node can always be safely
eliminated.
This commit is contained in:
Neil Jenkins 2023-02-01 14:19:52 +11:00
parent e4ae7c2b81
commit b85754ca50
1 changed files with 14 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
source/range/Boundaries.ts
View file

@ -44,10 +44,23 @@ const moveRangeBoundariesDownTree = (range: Range): void => {
let { startContainer, startOffset, endContainer, endOffset } = range; let { startContainer, startOffset, endContainer, endOffset } = range;
while (!(startContainer instanceof Text)) { while (!(startContainer instanceof Text)) {
let child = startContainer.childNodes[startOffset]; let child: ChildNode | null = startContainer.childNodes[startOffset];
if (!child || isLeaf(child)) { if (!child || isLeaf(child)) {
if (startOffset) { if (startOffset) {
child = startContainer.childNodes[startOffset - 1]; child = startContainer.childNodes[startOffset - 1];
let prev = child.previousSibling;
// If we have an empty text node next to another text node,
// just skip and remove it.
while (
child instanceof Text &&
!child.length &&
prev &&
prev instanceof Text
) {
child.remove();
child = prev;
continue;
}
if (child instanceof Text) { if (child instanceof Text) {
startContainer = child; startContainer = child;
startOffset = child.data.length; startOffset = child.data.length;