From 8ba6e1843a193a6e1040058dc0c74e5ad335e78d Mon Sep 17 00:00:00 2001
From: CJ van den Berg <cj@vdbonline.com>
Date: Wed, 26 Nov 2025 23:09:17 +0100
Subject: [PATCH] refactor: don't special case cursor rendering in inclusive
 mode

This is too confusing. Rendering cursors differently, only in inclusive
mode and only if there is an active selection is too confusing and pushes
a lot of edge cases into otherwise simple commands.

This will likely break a lot of the existing helix commands, but is better
in the long run to fix them anyway.
---
 src/tui/editor.zig | 19 ++-----------------
 1 file changed, 2 insertions(+), 17 deletions(-)

diff --git a/src/tui/editor.zig b/src/tui/editor.zig
index ddcda8f..83d2832 100644
--- a/src/tui/editor.zig
+++ b/src/tui/editor.zig
@@ -146,13 +146,6 @@ pub const CurSel = struct {
         };
     }
 
-    fn to_cursor_inclusive(self: *const Self, root: Buffer.Root, metrics: Buffer.Metrics) Cursor {
-        var cursor = self.cursor;
-        if (self.selection) |sel| if (!sel.is_reversed())
-            cursor.move_left(root, metrics) catch {};
-        return cursor;
-    }
-
     pub fn disable_selection(self: *Self, root: Buffer.Root, metrics: Buffer.Metrics) void {
         switch (tui.get_selection_style()) {
             .normal => self.disable_selection_normal(),
@@ -1181,26 +1174,18 @@ pub const Editor = struct {
     }
 
     fn render_cursors(self: *Self, theme: *const Widget.Theme, cell_map: CellMap) !void {
-        const style = tui.get_selection_style();
         const frame = tracy.initZone(@src(), .{ .name = "editor render cursors" });
         defer frame.deinit();
         if (tui.config().enable_terminal_cursor and tui.rdr().vx.caps.multi_cursor)
             tui.rdr().clear_all_multi_cursors() catch {};
         for (self.cursels.items[0 .. self.cursels.items.len - 1]) |*cursel_| if (cursel_.*) |*cursel| {
-            const cursor = self.get_rendered_cursor(style, cursel);
+            const cursor = cursel.cursor;
             try self.render_cursor_secondary(&cursor, theme, cell_map);
         };
-        const cursor = self.get_rendered_cursor(style, self.get_primary());
+        const cursor = self.get_primary().cursor;
         try self.render_cursor_primary(&cursor, theme, cell_map);
     }
 
-    fn get_rendered_cursor(self: *Self, style: anytype, cursel: anytype) Cursor {
-        return switch (style) {
-            .normal => cursel.cursor,
-            .inclusive => cursel.to_cursor_inclusive(self.buf_root() catch return cursel.cursor, self.metrics),
-        };
-    }
-
     fn render_cursor_primary(self: *Self, cursor: *const Cursor, theme: *const Widget.Theme, cell_map: CellMap) !void {
         if (!tui.is_mainview_focused() or !self.enable_terminal_cursor) {
             if (self.screen_cursor(cursor)) |pos| {