mirror of
https://github.com/zed-industries/zed.git
synced 2025-01-12 21:32:40 +00:00
Fix vim-related panic
This commit is contained in:
parent
5c8ff05577
commit
417f28effe
1 changed files with 29 additions and 1 deletions
|
@ -34,11 +34,11 @@ fn focused(EditorFocused(editor): &EditorFocused, cx: &mut AppContext) {
|
||||||
fn blurred(EditorBlurred(editor): &EditorBlurred, cx: &mut AppContext) {
|
fn blurred(EditorBlurred(editor): &EditorBlurred, cx: &mut AppContext) {
|
||||||
editor.window().update(cx, |cx| {
|
editor.window().update(cx, |cx| {
|
||||||
Vim::update(cx, |vim, cx| {
|
Vim::update(cx, |vim, cx| {
|
||||||
vim.clear_operator(cx);
|
|
||||||
vim.workspace_state.recording = false;
|
vim.workspace_state.recording = false;
|
||||||
vim.workspace_state.recorded_actions.clear();
|
vim.workspace_state.recorded_actions.clear();
|
||||||
if let Some(previous_editor) = vim.active_editor.clone() {
|
if let Some(previous_editor) = vim.active_editor.clone() {
|
||||||
if previous_editor == editor.clone() {
|
if previous_editor == editor.clone() {
|
||||||
|
vim.clear_operator(cx);
|
||||||
vim.active_editor = None;
|
vim.active_editor = None;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -60,3 +60,31 @@ fn released(EditorReleased(editor): &EditorReleased, cx: &mut AppContext) {
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[cfg(test)]
|
||||||
|
mod test {
|
||||||
|
use crate::{test::VimTestContext, Vim};
|
||||||
|
use editor::Editor;
|
||||||
|
use gpui::View;
|
||||||
|
use language::Buffer;
|
||||||
|
|
||||||
|
// regression test for blur called with a different active editor
|
||||||
|
#[gpui::test]
|
||||||
|
async fn test_blur_focus(cx: &mut gpui::TestAppContext) {
|
||||||
|
let mut cx = VimTestContext::new(cx, true).await;
|
||||||
|
|
||||||
|
let buffer = cx.add_model(|_| Buffer::new(0, 0, "a = 1\nb = 2\n"));
|
||||||
|
let window2 = cx.add_window(|cx| Editor::for_buffer(buffer, None, cx));
|
||||||
|
let editor2 = cx.read(|cx| window2.root(cx)).unwrap();
|
||||||
|
|
||||||
|
cx.update(|cx| {
|
||||||
|
let vim = Vim::read(cx);
|
||||||
|
assert_eq!(vim.active_editor.unwrap().id(), editor2.id())
|
||||||
|
});
|
||||||
|
|
||||||
|
// no panic when blurring an editor in a different window.
|
||||||
|
cx.update_editor(|editor1, cx| {
|
||||||
|
editor1.focus_out(cx.handle().into_any(), cx);
|
||||||
|
});
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
Loading…
Reference in a new issue