kc/helix
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 3

fix: render rulers before the cursor

Browse source 
Render rulers before the cursor to ensure that the cursor, when over
a ruler, is not hidden from view. Without this, you typically end up
with 1) foreground text that is the same as the background if the
ruler doesn't already have a foreground and 2) no visible cursor,
because the ruler's background color took precedence. By moving the
rulers before the cursor, this ensures that the theme is still rendered
more or less the way one would visually expect things to turn out.
This commit is contained in:
Noel Cower 2025-03-07 12:42:41 -05:00 • committed by Michael Davis
parent 8da226f0b4
commit 2d3b75a8c5
No known key found for this signature in database
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
helix-term/src/ui/editor.rs
View file

@ -176,6 +176,8 @@ impl EditorView {
);
}
Self::render_rulers(editor, doc, view, inner, surface, theme);
let primary_cursor = doc
.selection(view.id)
.primary()
@ -210,7 +212,6 @@ impl EditorView {
theme,
decorations,
);
Self::render_rulers(editor, doc, view, inner, surface, theme);
// if we're not at the edge of the screen, draw a right border
if viewport.right() != view.area.right() {