curioustorvald/tsvm
1
0
Fork 0
mirror of https://github.com/curioustorvald/tsvm.git synced 2026-03-07 19:51:51 +09:00
Code Issues Packages Projects Releases Wiki Activity

edit.js: minor asthetic fix

Browse Source
This commit is contained in:
minjaesong
2021-04-22 11:15:51 +09:00
parent 320aea286d
commit 8997b83114
1 changed files with 25 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

41
assets/disk0/tvdos/bin/edit.js
View File

@@ -333,19 +333,23 @@ function cursorMoveRelative(odx, ody) {
if (nextCol - scrollHor > visible) {
scrollHor = nextCol - visible;
// scroll to the right?
if (scrollHor > textbuffer[nextRow].length - paintWidth + scrollHorPeek)
// to prevent overscrolling that might happen after some complex navigation, AND
// to make sure text cursor to be placed at the right end of the screen where "more line arrow"
// goes which also makes editing field 1 character wider
scrollHor = textbuffer[nextRow].length - paintWidth + scrollHorPeek;
}
else if (nextCol - scrollHor < 0 + scrollHorPeek) {
scrollHor = nextCol - scrollHorPeek; // nextCol is less than zero
// scroll to the left?
if (scrollHor <= -1)
scrollHor = 0;
}
// NOTE: this scroll clamping is moved outside of go-left/go-right if-statements above because
// vertical movements can disrupt horizontal scrolls as well due to the cursoringCol variable
// scroll to the left?
if (scrollHor <= -1)
scrollHor = 0;
// scroll to the right?
else if (scrollHor > textbuffer[nextRow].length - paintWidth + scrollHorPeek)
// to prevent overscrolling that might happen after some complex navigation, AND
// to make sure text cursor to be placed at the right end of the screen where "more line arrow"
// goes which also makes editing field 1 character wider
scrollHor = textbuffer[nextRow].length - paintWidth + scrollHorPeek;
}
// update vertical scroll stats
@@ -354,17 +358,22 @@ function cursorMoveRelative(odx, ody) {
if (nextRow - scroll > visible) {
scroll = nextRow - visible;
// scroll to the bottom?
if (scroll > textbuffer.length - paintHeight)
// to make sure not show buncha empty lines
scroll = textbuffer.length - paintHeight;
}
else if (nextRow - scroll < 0 + scrollPeek) {
scroll = nextRow - scrollPeek; // nextRow is less than zero
// scroll to the top?
if (scroll <= -1)
scroll = 0; // scroll of -1 would result to show "Line 0" on screen
}
// NOTE: future-proofing here -- scroll clamping is moved outside of go-up/go-down
// if-statements above because horizontal movements can disrupt vertical scrolls as well because
// "normally" when you go right at the end of the line, you appear at the start of the next line
// scroll to the bottom?
if (scroll > textbuffer.length - paintHeight)
// to make sure not show buncha empty lines
scroll = textbuffer.length - paintHeight;
// scroll to the top?
else if (scroll <= -1)
scroll = 0; // scroll of -1 would result to show "Line 0" on screen
}