From e9c0034f4d3ee003149fe5454942bea7ff3d98be Mon Sep 17 00:00:00 2001 From: Kirill Chibisov Date: Sun, 27 Sep 2020 02:51:31 +0300 Subject: Fix selection incorrectly expanding when scrolled in history When doing selection expansion we were checking for wide char flags on a cells from the bottom of the terminal instead of in a current viewport when scrolled up in history, which was leading to expanding more than needed if we had wide chars on the same viewport cell, but in the bottom of the terminal. Fixes #4257. --- alacritty_terminal/src/term/mod.rs | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) (limited to 'alacritty_terminal/src') diff --git a/alacritty_terminal/src/term/mod.rs b/alacritty_terminal/src/term/mod.rs index d61cf7e3..00726dad 100644 --- a/alacritty_terminal/src/term/mod.rs +++ b/alacritty_terminal/src/term/mod.rs @@ -210,23 +210,28 @@ impl<'a, C> RenderableCellsIter<'a, C> { } let num_cols = self.grid.cols(); - let cell = self.grid[&point]; + + // Convert to absolute coordinates to adjust for the display offset. + let buffer_point = self.grid.visible_to_buffer(point); + let cell = self.grid[buffer_point]; // Check if wide char's spacers are selected. if cell.flags.contains(Flags::WIDE_CHAR) { let prev = point.sub(num_cols, 1); + let buffer_prev = self.grid.visible_to_buffer(prev); let next = point.add(num_cols, 1); // Check trailing spacer. selection.contains(next.col, next.line) // Check line-wrapping, leading spacer. - || (self.grid[&prev].flags.contains(Flags::LEADING_WIDE_CHAR_SPACER) + || (self.grid[buffer_prev].flags.contains(Flags::LEADING_WIDE_CHAR_SPACER) && selection.contains(prev.col, prev.line)) } else if cell.flags.contains(Flags::WIDE_CHAR_SPACER) { // Check if spacer's wide char is selected. let prev = point.sub(num_cols, 1); + let buffer_prev = self.grid.visible_to_buffer(prev); - if self.grid[&prev].flags.contains(Flags::WIDE_CHAR) { + if self.grid[buffer_prev].flags.contains(Flags::WIDE_CHAR) { // Check previous cell for trailing spacer. self.is_selected(prev) } else { -- cgit