extern/heynote
1
0
Fork 0
mirror of https://github.com/heyman/heynote.git synced 2024-11-22 07:54:11 +01:00
Code Issues Packages Projects Releases Wiki Activity

When moving all lines from the last block upwards, make sure we don't create a block with empty block content (which will create an invalid syntax tree)

Browse Source 
#windows-build
This commit is contained in:
Jonatan Heyman 2023-01-24 16:34:16 +01:00
parent a3545f5b52
commit 428a3172ab
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/editor/block/move-lines.js
View File

@ -42,7 +42,8 @@ function moveLine(state, dispatch, forward) {
// if the whole selection is a block (surrounded by separators) we need to add an extra line break between the separators that'll
// get stacked on top of each other, since we'll otherwise create two separators with only a single line break between them which
// the syntax parser won't be able to parse (since a valid separator needs one line break on each side)
let blockSurroundedBySeparators = previousLine !== null && previousLine.text.match(tokenRegEx) && nextLine.text.match(tokenRegEx)
let nextLineIsSeparator = nextLine.text.match(tokenRegEx)
let blockSurroundedBySeparators = previousLine !== null && previousLine.text.match(tokenRegEx) && nextLineIsSeparator
let size = nextLine.length + 1;
if (forward) {
if (blockSurroundedBySeparators) {
@ -55,7 +56,7 @@ function moveLine(state, dispatch, forward) {
ranges.push(EditorSelection.range(Math.min(state.doc.length, r.anchor + size), Math.min(state.doc.length, r.head + size)));
}
else {
if (blockSurroundedBySeparators) {
if (blockSurroundedBySeparators || (previousLine === null && nextLineIsSeparator)) {
//size += 1
changes.push({ from: nextLine.from, to: block.from }, { from: block.to, insert: state.lineBreak + nextLine.text + state.lineBreak});
for (let r of block.ranges)