Skip to content

Commit

Permalink
editor: Fix highlight selection panic (#25141)
Browse files Browse the repository at this point in the history
Use the latest selection state for highlighting instead of the state
before debounce.

Release Notes:

- N/A
  • Loading branch information
0xtimsb authored and osiewicz committed Feb 19, 2025
1 parent a897be4 commit 1a79a56
Showing 1 changed file with 13 additions and 13 deletions.
26 changes: 13 additions & 13 deletions crates/editor/src/editor.rs
Original file line number Diff line number Diff line change
Expand Up @@ -4816,24 +4816,24 @@ impl Editor {
self.clear_background_highlights::<SelectedTextHighlight>(cx);
return;
}
if self.selections.count() != 1 || self.selections.line_mode {
self.clear_background_highlights::<SelectedTextHighlight>(cx);
return;
}
let selection = self.selections.newest::<Point>(cx);
if selection.is_empty() || selection.start.row != selection.end.row {
self.clear_background_highlights::<SelectedTextHighlight>(cx);
return;
}
let debounce = EditorSettings::get_global(cx).selection_highlight_debounce;
self.selection_highlight_task = Some(cx.spawn_in(window, |editor, mut cx| async move {
cx.background_executor()
.timer(Duration::from_millis(debounce))
.await;
let Some(matches_task) = editor
.read_with(&mut cx, |editor, cx| {
let Some(Some(matches_task)) = editor
.update_in(&mut cx, |editor, _, cx| {
if editor.selections.count() != 1 || editor.selections.line_mode {
editor.clear_background_highlights::<SelectedTextHighlight>(cx);
return None;
}
let selection = editor.selections.newest::<Point>(cx);
if selection.is_empty() || selection.start.row != selection.end.row {
editor.clear_background_highlights::<SelectedTextHighlight>(cx);
return None;
}
let buffer = editor.buffer().read(cx).snapshot(cx);
cx.background_spawn(async move {
Some(cx.background_spawn(async move {
let mut ranges = Vec::new();
let buffer_ranges =
vec![buffer.anchor_before(0)..buffer.anchor_after(buffer.len())];
Expand Down Expand Up @@ -4871,7 +4871,7 @@ impl Editor {
}
}
ranges
})
}))
})
.log_err()
else {
Expand Down

0 comments on commit 1a79a56

Please sign in to comment.