From 45f48fd6d08468fd2b1030d9d1da6df81709e071 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Morten=20Aslo-=C3=98stergaard?= <morten@itinstituttet.dk>
Date: Mon, 9 Nov 2026 09:10:03 +0100
Subject: [PATCH] =?UTF-8?q?chore:=20v0.11.0=20release=20prep=20=E2=80=94?=
 =?UTF-8?q?=20Recent=20button=20layout,=20Recent=20UI=20test=20on=20an=20u?=
 =?UTF-8?q?nlocked=20desktop?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- The 🕘 Recent button was the DockPanel's last child, so it filled the
  remaining toolbar width and sat stretched across the middle. New Session and
  Recent now share a left-aligned StackPanel.
- RecentSessionsTests looked for the ContextMenu popup only at the top level.
  On an unlocked desktop UI Automation lists it as a child of the main window
  instead; the test now looks in both places. 12/12 UI tests, 1183 unit tests.

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

diff --git a/src/CodeShellManager/MainWindow.xaml b/src/CodeShellManager/MainWindow.xaml
index 6b8ae85..6afcf02 100644
--- a/src/CodeShellManager/MainWindow.xaml
+++ b/src/CodeShellManager/MainWindow.xaml
@@ -217,11 +217,16 @@
                             Click="Layout_ThreeByThree_Click" FontSize="9" Padding="4,4"/>
                 </StackPanel>
 
-                <Button DockPanel.Dock="Left" AutomationProperties.AutomationId="NewSessionBtn" Content="＋ New Session" Style="{StaticResource ToolBtn}"
-                        Foreground="#a6e3a1" Click="NewSession_Click" Margin="0,0,4,0"/>
-                <Button x:Name="RecentSessionsBtn" DockPanel.Dock="Left" AutomationProperties.AutomationId="RecentSessionsBtn"
-                        Content="🕘 Recent ▾" Style="{StaticResource ToolBtn}" Click="RecentSessions_Click" Margin="0,0,4,0"
-                        ToolTip="Jump to a recently used session (Ctrl+Shift+E)"/>
+                <!-- Last child of the DockPanel fills the remaining width, so the left-hand
+                     buttons share a left-aligned StackPanel — a bare button here stretches
+                     across the middle of the toolbar. -->
+                <StackPanel Orientation="Horizontal" HorizontalAlignment="Left" VerticalAlignment="Center">
+                    <Button AutomationProperties.AutomationId="NewSessionBtn" Content="＋ New Session" Style="{StaticResource ToolBtn}"
+                            Foreground="#a6e3a1" Click="NewSession_Click" Margin="0,0,4,0"/>
+                    <Button x:Name="RecentSessionsBtn" AutomationProperties.AutomationId="RecentSessionsBtn"
+                            Content="🕘 Recent ▾" Style="{StaticResource ToolBtn}" Click="RecentSessions_Click" Margin="0,0,4,0"
+                            ToolTip="Jump to a recently used session (Ctrl+Shift+E)"/>
+                </StackPanel>
             </DockPanel>
         </Border>
 
diff --git a/tests/CodeShellManager.UITests/RecentSessionsTests.cs b/tests/CodeShellManager.UITests/RecentSessionsTests.cs
index c5f5b06..9c3d178 100644
--- a/tests/CodeShellManager.UITests/RecentSessionsTests.cs
+++ b/tests/CodeShellManager.UITests/RecentSessionsTests.cs
@@ -20,11 +20,13 @@ public sealed class RecentSessionsTests : IClassFixture<AppFixture>
     private MenuItem[] OpenRecentMenu()
     {
         AppActions.Press(AppActions.WaitForElement(_f.MainWindow, "RecentSessionsBtn"));
-        // A ContextMenu is a popup: UI Automation lists it as an unnamed top-level window of
-        // the app's process with the Menu inside — not under the main window.
+        // A ContextMenu is a popup: an unnamed Window with the Menu inside. UI Automation lists
+        // it either at the top level of the app's process (seen on a locked desktop) or as a
+        // child of the main window (seen on an unlocked one), so look in both places.
         var menu = Retry.WhileNull(
             () => _f.Automation.GetDesktop()
                       .FindAllChildren(cf => cf.ByProcessId(_f.App.ProcessId))
+                      .SelectMany(w => w.FindAllChildren(cf => cf.ByControlType(ControlType.Window)).Prepend(w))
                       .Select(w => w.FindFirstChild(cf => cf.ByControlType(ControlType.Menu)))
                       .FirstOrDefault(m => m != null),
             TimeSpan.FromSeconds(5)).Result