From ab7cf50857ec173578affa7b6beb35fa0350635a Mon Sep 17 00:00:00 2001 From: Anthony Eid <56899983+Anthony-Eid@users.noreply.github.com> Date: Fri, 27 Feb 2026 14:24:26 +0100 Subject: [PATCH] Revert "sidebar: Zoom temporarily hides sidebar (#50088)" (#50297) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This reverts commit aa3a12b566, because we no longer want a zoomed item to block the sidebar. This is because the sidebar is used for navigation and it interrupts a user's flow if they're scrolling through their active workspaces, and the sidebar just vanishes. ### Before image ### After Screenshot 2026-02-27 at 2 10
24 PM Before you mark this PR as ready for review, make sure that you have: - [x] Added a solid test coverage and/or screenshots from doing manual testing - [x] Done a self-review taking into account security and performance aspects - [x] Aligned any UI changes with the [UI checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist) Release Notes: - N/A --- crates/workspace/src/multi_workspace.rs | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/crates/workspace/src/multi_workspace.rs b/crates/workspace/src/multi_workspace.rs index d58101f8b5c266fafa2120d8fe58634dc2414762..cd77f4fe30461b5f726c3bcd2f5f78b561e4d415 100644 --- a/crates/workspace/src/multi_workspace.rs +++ b/crates/workspace/src/multi_workspace.rs @@ -672,7 +672,6 @@ impl MultiWorkspace { impl Render for MultiWorkspace { fn render(&mut self, window: &mut Window, cx: &mut Context) -> impl IntoElement { let multi_workspace_enabled = self.multi_workspace_enabled(cx); - let is_zoomed = self.workspace().read(cx).zoomed_item().is_some(); let sidebar: Option = if multi_workspace_enabled && self.sidebar_open { self.sidebar.as_ref().map(|sidebar_handle| { @@ -784,14 +783,13 @@ impl Render for MultiWorkspace { .flex_1() .size_full() .overflow_hidden() - .when(is_zoomed, |this| this.absolute().inset_0()) .child(self.workspace().clone()), ) .child(self.workspace().read(cx).modal_layer.clone()), window, cx, Tiling { - left: multi_workspace_enabled && self.sidebar_open && !is_zoomed, + left: multi_workspace_enabled && self.sidebar_open, ..Tiling::default() }, )