neovim

commit f32557ca679cbb1d7de52ab54dc35585af9ab9d0
parent 0a0962a2e8d9ffa5f03e492fa293e964650a965e
Author: Gregory Anders <8965202+gpanders@users.noreply.github.com>
Date:   Fri,  2 Aug 2024 13:00:04 -0500

fix(tui): reset active attr ID when OSC 8 sequence is terminated (#29960)

When the cursor is moved we terminate any active OSC 8 sequences to
prevent the sequence from inadvertently spanning regions it is not meant
to span. However, if we do not also reset the TUI's active attr id
(print_attr_id) then the TUI does not "know" that it's current attribute
set has changed. When cursor_goto is called to wrap a line, the TUI does
not recompute the attributes so the OSC 8 sequence is not restarted
again.

When we terminate an OSC 8 sequence before moving the cursor, also reset
the active attr id so that the attributes are recomputed for URLs.
Diffstat:
M
src/nvim/tui/tui.c
 | 
1
+

1 file changed, 1 insertion(+), 0 deletions(-)
diff --git a/src/nvim/tui/tui.c b/src/nvim/tui/tui.c
@@ -888,6 +888,7 @@ static void cursor_goto(TUIData *tui, int row, int col)
   if (tui->url >= 0) {
     out(tui, S_LEN("\x1b]8;;\x1b\\"));
     tui->url = -1;
+    tui->print_attr_id = -1;
   }
 
   if (0 == row && 0 == col) {