Fix URL marking + scrolling

The ranges have to use on screen line numbers not line->ynum
Fixes #2969

Author: kovidgoyal

kitty/screen.c

@@ -2509,15 +2509,15 @@ screen_mark_url(Screen *self, index_type start_x, index_type start_y, index_type
 }
 
 static bool
-mark_hyperlinks_in_line(Screen *self, Line *line, hyperlink_id_type id) {
+mark_hyperlinks_in_line(Screen *self, Line *line, hyperlink_id_type id, index_type y) {
     index_type start = 0;
     bool found = false;
     bool in_range = false;
     for (index_type x = 0; x < line->xnum; x++) {
         bool has_hyperlink = line->cpu_cells[x].hyperlink_id == id;
         if (in_range) {
             if (!has_hyperlink) {
-                add_url_range(self, start, line->ynum, x - 1, line->ynum);
+                add_url_range(self, start, y, x - 1, y);
                 in_range = false;
                 start = 0;
             }
@@ -2528,7 +2528,7 @@ mark_hyperlinks_in_line(Screen *self, Line *line, hyperlink_id_type id) {
             }
         }
     }
-    if (in_range) add_url_range(self, start, line->ynum, self->columns - 1, line->ynum);
+    if (in_range) add_url_range(self, start, y, self->columns - 1, y);
     return found;
 }
 
@@ -2559,15 +2559,15 @@ screen_mark_hyperlink(Screen *self, index_type x, index_type y) {
     if (!id) return 0;
     index_type ypos = y, last_marked_line = y;
     do {
-        if (mark_hyperlinks_in_line(self, line, id)) last_marked_line = ypos;
+        if (mark_hyperlinks_in_line(self, line, id, ypos)) last_marked_line = ypos;
         if (ypos == 0) break;
         ypos--;
         line = screen_visual_line(self, ypos);
     } while (last_marked_line - ypos < 5);
     ypos = y + 1; last_marked_line = y;
     while (ypos < self->lines - 1 && ypos - last_marked_line < 5) {
         line = screen_visual_line(self, ypos);
-        if (mark_hyperlinks_in_line(self, line, id)) last_marked_line = ypos;
+        if (mark_hyperlinks_in_line(self, line, id, ypos)) last_marked_line = ypos;
         ypos++;
     }
     if (self->url_ranges.count > 1) sort_ranges(self, &self->url_ranges);