Fix incorrect cell foreground when clearing screen
This fixes a bug that would clear the cells with the current template cell with just the `flags` reset, to make sure the colors are correct. However, the cell foreground was not reset, leading to cells counting as occupied when resizing. With this change both cell flags and foreground color are ignored when clearing both the whole screen and inside the line, allowing us to accurately keep track of cell occupation. Fixes #2866.
This commit is contained in:
parent
b47a88b142
commit
93e87eb0f1
|
@ -398,11 +398,8 @@ impl<N: Notify> Processor<N> {
|
|||
font_size: &mut self.font_size,
|
||||
config: &mut self.config,
|
||||
};
|
||||
let mut processor = input::Processor::new(
|
||||
context,
|
||||
&self.display.urls,
|
||||
&self.display.highlighted_url,
|
||||
);
|
||||
let mut processor =
|
||||
input::Processor::new(context, &self.display.urls, &self.display.highlighted_url);
|
||||
|
||||
for event in event_queue.drain(..) {
|
||||
Processor::handle_event(event, &mut processor);
|
||||
|
|
|
@ -20,9 +20,9 @@
|
|||
//! determine what to do when a non-modifier key is pressed.
|
||||
use std::borrow::Cow;
|
||||
use std::cmp::min;
|
||||
use std::cmp::Ordering;
|
||||
use std::marker::PhantomData;
|
||||
use std::time::Instant;
|
||||
use std::cmp::Ordering;
|
||||
|
||||
use glutin::event::{
|
||||
ElementState, KeyboardInput, ModifiersState, MouseButton, MouseScrollDelta, TouchPhase,
|
||||
|
@ -250,11 +250,7 @@ impl From<MouseState> for CursorIcon {
|
|||
}
|
||||
|
||||
impl<'a, T: EventListener, A: ActionContext<T>> Processor<'a, T, A> {
|
||||
pub fn new(
|
||||
ctx: A,
|
||||
urls: &'a Urls,
|
||||
highlighted_url: &'a Option<Url>,
|
||||
) -> Self {
|
||||
pub fn new(ctx: A, urls: &'a Urls, highlighted_url: &'a Option<Url>) -> Self {
|
||||
Self { ctx, urls, highlighted_url, _phantom: Default::default() }
|
||||
}
|
||||
|
||||
|
@ -646,10 +642,12 @@ impl<'a, T: EventListener, A: ActionContext<T>> Processor<'a, T, A> {
|
|||
let new_icon = match current_lines.cmp(&new_lines) {
|
||||
Ordering::Less => CursorIcon::Default,
|
||||
Ordering::Equal => CursorIcon::Hand,
|
||||
Ordering::Greater => if mouse_mode {
|
||||
CursorIcon::Default
|
||||
} else {
|
||||
CursorIcon::Text
|
||||
Ordering::Greater => {
|
||||
if mouse_mode {
|
||||
CursorIcon::Default
|
||||
} else {
|
||||
CursorIcon::Text
|
||||
}
|
||||
},
|
||||
};
|
||||
|
||||
|
|
|
@ -1661,8 +1661,7 @@ impl<T: EventListener> ansi::Handler for Term<T> {
|
|||
#[inline]
|
||||
fn clear_line(&mut self, mode: ansi::LineClearMode) {
|
||||
trace!("Clearing line: {:?}", mode);
|
||||
let mut template = self.cursor.template;
|
||||
template.flags ^= template.flags;
|
||||
let template = Cell { bg: self.cursor.template.bg, ..Cell::default() };
|
||||
|
||||
let col = self.cursor.point.col;
|
||||
|
||||
|
@ -1725,8 +1724,7 @@ impl<T: EventListener> ansi::Handler for Term<T> {
|
|||
#[inline]
|
||||
fn clear_screen(&mut self, mode: ansi::ClearMode) {
|
||||
trace!("Clearing screen: {:?}", mode);
|
||||
let mut template = self.cursor.template;
|
||||
template.flags ^= template.flags;
|
||||
let template = Cell { bg: self.cursor.template.bg, ..Cell::default() };
|
||||
|
||||
// Remove active selections
|
||||
self.grid.selection = None;
|
||||
|
|
File diff suppressed because one or more lines are too long
File diff suppressed because one or more lines are too long
File diff suppressed because one or more lines are too long
Loading…
Reference in New Issue