From f7d46cf7d02c88d3d71ec495a31d7f19bd5eb96b Mon Sep 17 00:00:00 2001
From: Cameron Mcloughlin <cameron.studdstreet@gmail.com>
Date: Tue, 21 Apr 2026 17:09:55 +0100
Subject: [PATCH] sidebar: Move to new workspace non-Wayland (#54055)

We were just creating a window without showing it. However, wayland does
not respect this setting, so it seemed to work.

Now we actually activate the window :)

Rather than changing the single call-site, we always do this when
calling `Workspace::new_local`, since there were no code paths in the
repo that pass `NewWindow` without immediately activating it

Release Notes:

- N/A or Added/Fixed/Improved ...
---
 crates/workspace/src/workspace.rs | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/crates/workspace/src/workspace.rs b/crates/workspace/src/workspace.rs
index 40d6a36969267efb9e8de4e1207a9c95878e84ed..b4afd209216101dcefebb2103436872c0a4084d3 100644
--- a/crates/workspace/src/workspace.rs
+++ b/crates/workspace/src/workspace.rs
@@ -2092,6 +2092,15 @@ impl Workspace {
                     });
                 })
                 .log_err();
+
+            if open_mode == OpenMode::NewWindow {
+                window
+                    .update(cx, |_, window, _cx| {
+                        window.activate_window();
+                    })
+                    .log_err();
+            }
+
             Ok(OpenResult {
                 window,
                 workspace,