From 81b92ce2aef36e81caae4873d49ecbb0aa762b1d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Morten=20Aslo-=C3=98stergaard?= <morten@itinstituttet.dk>
Date: Thu, 29 Oct 2026 15:04:18 +0100
Subject: [PATCH] feat(toolbar): move "RC All" into the sidebar's Bulk actions
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The toolbar carried a large "⊕ RC All" button directly beside "＋ New
Session". It is rarely wanted, easy to hit by mistake, and a stray click
types /remote-control into every live session at once. It doesn't earn
toolbar space.

It now lives under Bulk actions in the sidebar menu as "Remote control
all", first in the list. It does exactly what the button did: sends
/remote-control to every live session and clears their alert state. Like
its Restart / Sleep / Close all siblings, it is disabled when no session is
live. It deliberately has no confirmation, matching the old button and the
existing per-group "Remote control all in group".

The BroadcastRemoteControl_Click handler is removed with the button. No UI
test used its BroadcastBtn automation id; it is only named in two April
planning docs under docs/superpowers/, left as the historical record.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NsYm9iLc2aRnRDfV2uZRQX
---
 src/CodeShellManager/MainWindow.xaml    |  3 ---
 src/CodeShellManager/MainWindow.xaml.cs | 25 ++++++++++++++++---------
 2 files changed, 16 insertions(+), 12 deletions(-)

diff --git a/src/CodeShellManager/MainWindow.xaml b/src/CodeShellManager/MainWindow.xaml
index c324731..c517e0e 100644
--- a/src/CodeShellManager/MainWindow.xaml
+++ b/src/CodeShellManager/MainWindow.xaml
@@ -218,9 +218,6 @@
 
                 <Button DockPanel.Dock="Left" AutomationProperties.AutomationId="NewSessionBtn" Content="＋ New Session" Style="{StaticResource ToolBtn}"
                         Foreground="#a6e3a1" Click="NewSession_Click" Margin="0,0,4,0"/>
-                <Button DockPanel.Dock="Left" AutomationProperties.AutomationId="BroadcastBtn" Content="⊕ RC All" Style="{StaticResource ToolBtn}"
-                        Foreground="#89b4fa" Click="BroadcastRemoteControl_Click" Margin="0,0,4,0"
-                        ToolTip="Send /remote-control to all sessions"/>
             </DockPanel>
         </Border>
 
diff --git a/src/CodeShellManager/MainWindow.xaml.cs b/src/CodeShellManager/MainWindow.xaml.cs
index 6d821de..d0bb6c3 100644
--- a/src/CodeShellManager/MainWindow.xaml.cs
+++ b/src/CodeShellManager/MainWindow.xaml.cs
@@ -729,15 +729,6 @@ private async Task InitDatabaseAsync()
 
     private void NewSession_Click(object sender, RoutedEventArgs e) => OpenNewSessionDialog();
 
-    private void BroadcastRemoteControl_Click(object sender, RoutedEventArgs e)
-    {
-        foreach (var session in _vm.Sessions)
-        {
-            session.Bridge?.SendToTerminal("/remote-control\r");
-            session.AlertDetector?.NotifyUserInteracted();
-        }
-    }
-
     private void OpenNewSessionDialog(string defaultFolder = "")
         => OpenNewSessionDialogCore(defaultFolder, parent: null);
 
@@ -2440,6 +2431,21 @@ private System.Windows.Controls.ContextMenu BuildSidebarQuickMenu()
 
         // Global bulk actions — operate on every live or dormant session, not just one group.
         var bulkActions = new System.Windows.Controls.MenuItem { Header = "Bulk actions" };
+
+        // Lived on the toolbar as a large "⊕ RC All" button next to "＋ New Session", where it
+        // was easy to hit by mistake for something rarely wanted. Mirrors the per-group
+        // "Remote control all in group" entry.
+        var remoteControlAll = new System.Windows.Controls.MenuItem { Header = "Remote control all" };
+        remoteControlAll.Click += (_, _) =>
+        {
+            foreach (var vm in _vm.Sessions.ToList())
+            {
+                vm.Bridge?.SendToTerminal("/remote-control\r");
+                vm.AlertDetector?.NotifyUserInteracted();
+            }
+        };
+        bulkActions.Items.Add(remoteControlAll);
+
         var wakeAllDormant = new System.Windows.Controls.MenuItem { Header = "Wake all dormant" };
         wakeAllDormant.Click += async (_, _) =>
         {
@@ -2562,6 +2568,7 @@ private System.Windows.Controls.ContextMenu BuildSidebarQuickMenu()
 
             int liveCount = _vm.Sessions.Count;
             int dormantCount = _sessionManager.Sessions.Count(s => s.IsDormant);
+            remoteControlAll.IsEnabled = liveCount > 0;
             wakeAllDormant.IsEnabled = dormantCount > 0;
             restartAllGlobal.IsEnabled = liveCount > 0;
             sleepAllGlobal.IsEnabled = liveCount > 0;