Yehowshua/ZIm
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Restore editor state on reopen (#27672)

Browse source 
Closes https://github.com/zed-industries/zed/issues/11626
Part of https://github.com/zed-industries/zed/issues/12853

`"restore_on_file_reopen": true` in workspace settings can now be used
to enable and disable editor data between file reopens in the same pane:


https://github.com/user-attachments/assets/8d938ee1-d854-42a8-bbc3-2a4e4d7d5933

The settings are generic and panes' data store can be extended for
further entities, beyond editors.

---------------
Impl details: 

Currently, the project entry IDs seem to be stable across file reopens,
unlike BufferIds, so those were used.
Originally, the DB data was considered over in-memory one as editors
serialize their state anyway, but managing and exposing PaneIds out of
the DB is quite tedious and joining the DB data otherwise is not
possible.


Release Notes:

- Started to restore editor state on reopen
This commit is contained in:
Kirill Bulatov 2025-03-29 00:04:16 +02:00 • committed by GitHub
parent bbd1e628f0
commit e11e7df724
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
13 changed files with 611 additions and 48 deletions
Download patch file Download diff file Expand all files Collapse all files

392
crates/editor/src/editor_tests.rs
View file

@ -52,7 +52,7 @@ use util::{
};
use workspace::{
item::{FollowEvent, FollowableItem, Item, ItemHandle},
NavigationEntry, ViewId,
CloseAllItems, CloseInactiveItems, NavigationEntry, ViewId,
};
#[gpui::test]
@ -18284,6 +18284,396 @@ async fn test_apply_code_lens_actions_with_commands(cx: &mut gpui::TestAppContex
});
}
#[gpui::test]
async fn test_editor_restore_data_different_in_panes(cx: &mut TestAppContext) {
init_test(cx, |_| {});
let fs = FakeFs::new(cx.executor());
let main_text = r#"fn main() {
println!("1");
println!("2");
println!("3");
println!("4");
println!("5");
}"#;
let lib_text = "mod foo {}";
fs.insert_tree(
path!("/a"),
json!({
"lib.rs": lib_text,
"main.rs": main_text,
}),
)
.await;
let project = Project::test(fs, [path!("/a").as_ref()], cx).await;
let (workspace, cx) =
cx.add_window_view(|window, cx| Workspace::test_new(project.clone(), window, cx));
let worktree_id = workspace.update(cx, |workspace, cx| {
workspace.project().update(cx, |project, cx| {
project.worktrees(cx).next().unwrap().read(cx).id()
})
});
let expected_ranges = vec![
Point::new(0, 0)..Point::new(0, 0),
Point::new(1, 0)..Point::new(1, 1),
Point::new(2, 0)..Point::new(2, 2),
Point::new(3, 0)..Point::new(3, 3),
];
let pane_1 = workspace.update(cx, |workspace, _| workspace.active_pane().clone());
let editor_1 = workspace
.update_in(cx, |workspace, window, cx| {
workspace.open_path(
(worktree_id, "main.rs"),
Some(pane_1.downgrade()),
true,
window,
cx,
)
})
.unwrap()
.await
.downcast::<Editor>()
.unwrap();
pane_1.update(cx, |pane, cx| {
let open_editor = pane.active_item().unwrap().downcast::<Editor>().unwrap();
open_editor.update(cx, |editor, cx| {
assert_eq!(
editor.display_text(cx),
main_text,
"Original main.rs text on initial open",
);
assert_eq!(
editor
.selections
.all::<Point>(cx)
.into_iter()
.map(|s| s.range())
.collect::<Vec<_>>(),
vec![Point::zero()..Point::zero()],
"Default selections on initial open",
);
})
});
editor_1.update_in(cx, |editor, window, cx| {
editor.change_selections(None, window, cx, |s| {
s.select_ranges(expected_ranges.clone());
});
});
let pane_2 = workspace.update_in(cx, |workspace, window, cx| {
workspace.split_pane(pane_1.clone(), SplitDirection::Right, window, cx)
});
let editor_2 = workspace
.update_in(cx, |workspace, window, cx| {
workspace.open_path(
(worktree_id, "main.rs"),
Some(pane_2.downgrade()),
true,
window,
cx,
)
})
.unwrap()
.await
.downcast::<Editor>()
.unwrap();
pane_2.update(cx, |pane, cx| {
let open_editor = pane.active_item().unwrap().downcast::<Editor>().unwrap();
open_editor.update(cx, |editor, cx| {
assert_eq!(
editor.display_text(cx),
main_text,
"Original main.rs text on initial open in another panel",
);
assert_eq!(
editor
.selections
.all::<Point>(cx)
.into_iter()
.map(|s| s.range())
.collect::<Vec<_>>(),
vec![Point::zero()..Point::zero()],
"Default selections on initial open in another panel",
);
})
});
editor_2.update_in(cx, |editor, window, cx| {
editor.fold_ranges(expected_ranges.clone(), false, window, cx);
});
let _other_editor_1 = workspace
.update_in(cx, |workspace, window, cx| {
workspace.open_path(
(worktree_id, "lib.rs"),
Some(pane_1.downgrade()),
true,
window,
cx,
)
})
.unwrap()
.await
.downcast::<Editor>()
.unwrap();
pane_1
.update_in(cx, |pane, window, cx| {
pane.close_inactive_items(&CloseInactiveItems::default(), window, cx)
.unwrap()
})
.await
.unwrap();
drop(editor_1);
pane_1.update(cx, |pane, cx| {
pane.active_item()
.unwrap()
.downcast::<Editor>()
.unwrap()
.update(cx, |editor, cx| {
assert_eq!(
editor.display_text(cx),
lib_text,
"Other file should be open and active",
);
});
assert_eq!(pane.items().count(), 1, "No other editors should be open");
});
let _other_editor_2 = workspace
.update_in(cx, |workspace, window, cx| {
workspace.open_path(
(worktree_id, "lib.rs"),
Some(pane_2.downgrade()),
true,
window,
cx,
)
})
.unwrap()
.await
.downcast::<Editor>()
.unwrap();
pane_2
.update_in(cx, |pane, window, cx| {
pane.close_inactive_items(&CloseInactiveItems::default(), window, cx)
.unwrap()
})
.await
.unwrap();
drop(editor_2);
pane_2.update(cx, |pane, cx| {
let open_editor = pane.active_item().unwrap().downcast::<Editor>().unwrap();
open_editor.update(cx, |editor, cx| {
assert_eq!(
editor.display_text(cx),
lib_text,
"Other file should be open and active in another panel too",
);
});
assert_eq!(
pane.items().count(),
1,
"No other editors should be open in another pane",
);
});
let _editor_1_reopened = workspace
.update_in(cx, |workspace, window, cx| {
workspace.open_path(
(worktree_id, "main.rs"),
Some(pane_1.downgrade()),
true,
window,
cx,
)
})
.unwrap()
.await
.downcast::<Editor>()
.unwrap();
let _editor_2_reopened = workspace
.update_in(cx, |workspace, window, cx| {
workspace.open_path(
(worktree_id, "main.rs"),
Some(pane_2.downgrade()),
true,
window,
cx,
)
})
.unwrap()
.await
.downcast::<Editor>()
.unwrap();
pane_1.update(cx, |pane, cx| {
let open_editor = pane.active_item().unwrap().downcast::<Editor>().unwrap();
open_editor.update(cx, |editor, cx| {
assert_eq!(
editor.display_text(cx),
main_text,
"Previous editor in the 1st panel had no extra text manipulations and should get none on reopen",
);
assert_eq!(
editor
.selections
.all::<Point>(cx)
.into_iter()
.map(|s| s.range())
.collect::<Vec<_>>(),
expected_ranges,
"Previous editor in the 1st panel had selections and should get them restored on reopen",
);
})
});
pane_2.update(cx, |pane, cx| {
let open_editor = pane.active_item().unwrap().downcast::<Editor>().unwrap();
open_editor.update(cx, |editor, cx| {
assert_eq!(
editor.display_text(cx),
r#"fn main() {
⋯rintln!("1");
⋯intln!("2");
⋯ntln!("3");
println!("4");
println!("5");
}"#,
"Previous editor in the 2nd pane had folds and should restore those on reopen in the same pane",
);
assert_eq!(
editor
.selections
.all::<Point>(cx)
.into_iter()
.map(|s| s.range())
.collect::<Vec<_>>(),
vec![Point::zero()..Point::zero()],
"Previous editor in the 2nd pane had no selections changed hence should restore none",
);
})
});
}
#[gpui::test]
async fn test_editor_does_not_restore_data_when_turned_off(cx: &mut TestAppContext) {
init_test(cx, |_| {});
let fs = FakeFs::new(cx.executor());
let main_text = r#"fn main() {
println!("1");
println!("2");
println!("3");
println!("4");
println!("5");
}"#;
let lib_text = "mod foo {}";
fs.insert_tree(
path!("/a"),
json!({
"lib.rs": lib_text,
"main.rs": main_text,
}),
)
.await;
let project = Project::test(fs, [path!("/a").as_ref()], cx).await;
let (workspace, cx) =
cx.add_window_view(|window, cx| Workspace::test_new(project.clone(), window, cx));
let worktree_id = workspace.update(cx, |workspace, cx| {
workspace.project().update(cx, |project, cx| {
project.worktrees(cx).next().unwrap().read(cx).id()
})
});
let pane = workspace.update(cx, |workspace, _| workspace.active_pane().clone());
let editor = workspace
.update_in(cx, |workspace, window, cx| {
workspace.open_path(
(worktree_id, "main.rs"),
Some(pane.downgrade()),
true,
window,
cx,
)
})
.unwrap()
.await
.downcast::<Editor>()
.unwrap();
pane.update(cx, |pane, cx| {
let open_editor = pane.active_item().unwrap().downcast::<Editor>().unwrap();
open_editor.update(cx, |editor, cx| {
assert_eq!(
editor.display_text(cx),
main_text,
"Original main.rs text on initial open",
);
})
});
editor.update_in(cx, |editor, window, cx| {
editor.fold_ranges(vec![Point::new(0, 0)..Point::new(0, 0)], false, window, cx);
});
cx.update_global(|store: &mut SettingsStore, cx| {
store.update_user_settings::<WorkspaceSettings>(cx, |s| {
s.restore_on_file_reopen = Some(false);
});
});
editor.update_in(cx, |editor, window, cx| {
editor.fold_ranges(
vec![
Point::new(1, 0)..Point::new(1, 1),
Point::new(2, 0)..Point::new(2, 2),
Point::new(3, 0)..Point::new(3, 3),
],
false,
window,
cx,
);
});
pane.update_in(cx, |pane, window, cx| {
pane.close_all_items(&CloseAllItems::default(), window, cx)
.unwrap()
})
.await
.unwrap();
pane.update(cx, |pane, _| {
assert!(pane.active_item().is_none());
});
cx.update_global(|store: &mut SettingsStore, cx| {
store.update_user_settings::<WorkspaceSettings>(cx, |s| {
s.restore_on_file_reopen = Some(true);
});
});
let _editor_reopened = workspace
.update_in(cx, |workspace, window, cx| {
workspace.open_path(
(worktree_id, "main.rs"),
Some(pane.downgrade()),
true,
window,
cx,
)
})
.unwrap()
.await
.downcast::<Editor>()
.unwrap();
pane.update(cx, |pane, cx| {
let open_editor = pane.active_item().unwrap().downcast::<Editor>().unwrap();
open_editor.update(cx, |editor, cx| {
assert_eq!(
editor.display_text(cx),
main_text,
"No folds: even after enabling the restoration, previous editor's data should not be saved to be used for the restoration"
);
})
});
}
fn empty_range(row: usize, column: usize) -> Range<DisplayPoint> {
let point = DisplayPoint::new(DisplayRow(row as u32), column as u32);
point..point