From 579dc927cc9d7631918f1b387cebc8c4e7ed7e8a Mon Sep 17 00:00:00 2001 From: Danilo Leal <67129314+danilo-leal@users.noreply.github.com> Date: Thu, 16 Apr 2026 07:16:58 -0300 Subject: [PATCH] sidebar: Fix ellipsis menu options (#54030) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Just a quick follow up to https://github.com/zed-industries/zed/pull/54025 where I incorrectly wrapped the "Remove Project" button in a conditional, when it should be available at all times. Screenshot 2026-04-16 at 12  00@2x Release Notes: - N/A --- crates/sidebar/src/sidebar.rs | 32 +++++++++++++------------------- 1 file changed, 13 insertions(+), 19 deletions(-) diff --git a/crates/sidebar/src/sidebar.rs b/crates/sidebar/src/sidebar.rs index 210fb699f0bde93a12ef045f342c455fdd856227..2d104afab82439272355cc250d776b051c862ed9 100644 --- a/crates/sidebar/src/sidebar.rs +++ b/crates/sidebar/src/sidebar.rs @@ -1751,25 +1751,19 @@ impl Sidebar { ) .selectable(!is_active); - menu.when(show_multi_project_entries, |menu| { - let project_group_key = project_group_key.clone(); - let multi_workspace = multi_workspace.clone(); - menu.separator() - .entry("Remove Project", None, move |window, cx| { - multi_workspace - .update(cx, |multi_workspace, cx| { - multi_workspace - .remove_project_group( - &project_group_key, - window, - cx, - ) - .detach_and_log_err(cx); - }) - .ok(); - weak_menu.update(cx, |_, cx| cx.emit(DismissEvent)).ok(); - }) - }) + let project_group_key = project_group_key.clone(); + let multi_workspace = multi_workspace.clone(); + menu.separator() + .entry("Remove Project", None, move |window, cx| { + multi_workspace + .update(cx, |multi_workspace, cx| { + multi_workspace + .remove_project_group(&project_group_key, window, cx) + .detach_and_log_err(cx); + }) + .ok(); + weak_menu.update(cx, |_, cx| cx.emit(DismissEvent)).ok(); + }) }); let this = this.clone();