From 2d3b75a8c591739d7541c05ff5c6b6e3dc63d012 Mon Sep 17 00:00:00 2001
From: Noel Cower <ncower@nil.dev>
Date: Fri, 7 Mar 2025 12:42:41 -0500
Subject: [PATCH] fix: render rulers before the cursor

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.
---
 helix-term/src/ui/editor.rs | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/helix-term/src/ui/editor.rs b/helix-term/src/ui/editor.rs
index d8500ed4d..6be565747 100644
--- a/helix-term/src/ui/editor.rs
+++ b/helix-term/src/ui/editor.rs
@@ -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() {