ilyakooo0/helix
1
0
Fork 0
mirror of https://github.com/ilyakooo0/helix.git synced 2024-10-26 17:23:34 +03:00
Code Issues Packages Projects Releases Wiki Activity

Fix phantom lines in some CRLF files.

Browse Source 
Fixes #415.  The issue was that cursor highlighting wasn't extending
to encompass the entire CRLF grapheme, and therefore ended up splitting
it.  This presumably was messing up other grapheme rendering as
well, and this fixes that as well.
This commit is contained in:
Nathan Vegdahl 2021-07-05 00:11:07 -07:00 committed by Ivan Tham
parent fc34efea12
commit 4952d6f801
1 changed files with 10 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
helix-term/src/ui/editor.rs
View File

@ -8,7 +8,7 @@
use helix_core::{
coords_at_pos,
graphemes::ensure_grapheme_boundary,
graphemes::{ensure_grapheme_boundary, next_grapheme_boundary},
syntax::{self, HighlightEvent},
LineEnding, Position, Range,
};
@ -187,19 +187,24 @@ pub fn render_buffer(
(cursor_scope, selection_scope)
};
let cursor_end = next_grapheme_boundary(text, range.head); // Used in every case below.
if range.head == range.anchor {
spans.push((cursor_scope, range.head..range.head + 1));
spans.push((cursor_scope, range.head..cursor_end));
continue;
}
let reverse = range.head < range.anchor;
if reverse {
spans.push((cursor_scope, range.head..range.head + 1));
spans.push((selection_scope, range.head + 1..range.anchor + 1));
spans.push((cursor_scope, range.head..cursor_end));
spans.push((
selection_scope,
cursor_end..next_grapheme_boundary(text, range.anchor),
));
} else {
spans.push((selection_scope, range.anchor..range.head));
spans.push((cursor_scope, range.head..range.head + 1));
spans.push((cursor_scope, range.head..cursor_end));
}
}