sidebar: Move to new workspace non-Wayland (#54055)

Cameron Mcloughlin created

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 ...

Change summary

crates/workspace/src/workspace.rs | 9 +++++++++
1 file changed, 9 insertions(+)

Detailed changes

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,