Fix invisible selection

This resolves a bug where the very first/last cell would still be
selected when both the start and the end were below/above the viewport.

1 files changed, 8 insertions, 2 deletions

diff --git a/alacritty_terminal/src/term/mod.rs b/alacritty_terminal/src/term/mod.rs
index 89c3723f..b5fced05 100644
--- a/alacritty_terminal/src/term/mod.rs
+++ b/alacritty_terminal/src/term/mod.rs
@@ -226,7 +226,6 @@ impl<'a, C> RenderableCellsIter<'a, C> {
         selection: Option<SelectionRange>,
     ) -> RenderableCellsIter<'b, C> {
         let grid = &term.grid;
-        let num_cols = grid.num_cols();
 
         let inner = grid.display_iter();
 
@@ -234,9 +233,16 @@ impl<'a, C> RenderableCellsIter<'a, C> {
             let (limit_start, limit_end) = if span.is_block {
                 (span.start.col, span.end.col)
             } else {
-                (Column(0), num_cols - 1)
+                (Column(0), grid.num_cols() - 1)
             };
 
+            // Do not render completely offscreen selection
+            let viewport_start = grid.display_offset();
+            let viewport_end = viewport_start + grid.num_lines().0;
+            if span.end.line >= viewport_end || span.start.line < viewport_start {
+                return None;
+            }
+
             // Get on-screen lines of the selection's locations
             let mut start = grid.clamp_buffer_to_visible(span.start);
             let mut end = grid.clamp_buffer_to_visible(span.end);