1
0
mirror of https://github.com/helix-editor/helix synced 2024-11-10 10:34:45 +01:00

Fix scrolloff at view bottom (#6142)

Fixes a regression introduced in #5420 where a scrolloff of `x - 1`
was used instead if `x` at the bottom of the screen. This was
especially problematic if the scrolloff was set to `0` in that case
the scrolloff behaved as tough set to `-1` and the cursor disappeared
from the view if scrolled to the botoom.
This commit is contained in:
Pascal Kuthe 2023-03-03 17:05:40 +01:00 committed by GitHub
parent 2d5577dbe6
commit 5c716af7a2
Signed by: GitHub
GPG Key ID: 4AEE18F83AFDEB23

@ -242,7 +242,7 @@ impl View {
at_top = true;
true
}
Some((visual_pos, _)) if visual_pos.row >= vertical_viewport_end - scrolloff => {
Some((visual_pos, _)) if visual_pos.row + scrolloff + 1 >= vertical_viewport_end => {
if CENTERING && visual_pos.row >= vertical_viewport_end {
// cursor out of view
return None;
@ -257,7 +257,7 @@ impl View {
let v_off = if at_top {
scrolloff as isize
} else {
viewport.height as isize - scrolloff as isize
viewport.height as isize - scrolloff as isize - 1
};
(offset.anchor, offset.vertical_offset) =
char_idx_at_visual_offset(doc_text, cursor, -v_off, 0, &text_fmt, &annotations);