imterah/element-portable
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

remove all non-first-br nodes on new empty line, not just first one

Browse source 
nextSibling returned null after calling removeNode,
so get the nextSibling first
This commit is contained in:
Bruno Windels 2019-05-28 10:01:16 +02:00
parent b2592583c4
commit 4fc23022f4
1 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
src/editor/render.js
View file

@ -61,12 +61,13 @@ export function renderModel(editor, model) {
let foundBR = false; let foundBR = false;
let partNode = lineContainer.firstChild; let partNode = lineContainer.firstChild;
while (partNode) { while (partNode) {
const nextNode = partNode.nextSibling;
if (!foundBR && partNode.tagName === "BR") { if (!foundBR && partNode.tagName === "BR") {
foundBR = true; foundBR = true;
} else { } else {
lineContainer.removeChild(partNode); lineContainer.removeChild(partNode);
} }
partNode = partNode.nextSibling; partNode = nextNode;
} }
if (!foundBR) { if (!foundBR) {
lineContainer.appendChild(document.createElement("br")); lineContainer.appendChild(document.createElement("br"));