Composer: neutral Stop fill + announce picker selection in a11y name

- Stop button now uses TextFillColorPrimaryBrush (theme-adaptive neutral)
  instead of the system accent; glyph uses SolidBackgroundFillColorBaseBrush
  so it stays legible against the inverted fill in light/dark. Keeps Send as
  the only accent control. Hover/pressed use TextFillColor secondary/tertiary.
- PickerButton folds the current selection into its AutomationName as
  "field: value", restoring the selection announcement the legacy ComboBox
  provided via SelectionPattern. Fixes the SessionTitleBehaviorProof UI test
  (Category=Accessibility) that read the selected route from the session
  selector; updated it to read the redesigned menu-flyout Button.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Author: karkarl

src/OpenClaw.Tray.WinUI/Chat/OpenClawComposer.cs

@@ -1276,6 +1276,14 @@ Element PickerButton(string label, string automationName, double? maxLabelWidth,
                     t.Foreground = mutedBrush;
                     t.VerticalAlignment = VerticalAlignment.Center;
                 });
+            // Fold the current selection into the accessible name so assistive
+            // tech announces "<field>: <value>" (e.g. "Session: <title>"), the
+            // way the legacy ComboBox surfaced its selected item. The visible
+            // chevron button only shows the value, so without this the current
+            // selection would be silent to screen readers.
+            var accessibleName = string.IsNullOrWhiteSpace(label)
+                ? automationName
+                : $"{automationName}: {label}";
             return Button(HStack(4, labelBlock, chevron))
                 .Set(b =>
                 {
@@ -1293,7 +1301,7 @@ Element PickerButton(string label, string automationName, double? maxLabelWidth,
                     .Set("ButtonBorderBrushPointerOver", new SolidColorBrush(Colors.Transparent))
                     .Set("ButtonBorderBrushPressed", new SolidColorBrush(Colors.Transparent)))
                 .WithFlyout(menu)
-                .AutomationName(automationName)
+                .AutomationName(accessibleName)
                 .SetToolTip(automationName);
         }
 
@@ -1458,8 +1466,9 @@ Element PickerButton(string label, string automationName, double? maxLabelWidth,
 
         // Stop button — occupies the SAME action slot as Send (identical size)
         // while the assistant is responding, so nothing in the toolbar shifts.
-        // Carries the system accent (Stop is the primary "act here" while a turn
-        // is active); red is reserved for genuine error states.
+        // Uses a neutral text-primary fill (theme-adaptive black/white); red is
+        // reserved for genuine error states. The glyph uses the base surface
+        // brush so it stays legible against the inverted fill in both themes.
         Element stopBtn = Empty();
         if (Props.TurnActive)
         {
@@ -1471,19 +1480,19 @@ Element PickerButton(string label, string automationName, double? maxLabelWidth,
                         t.FontFamily = FluentIconCatalog.SymbolThemeFontFamily;
                         t.FontSize = composerIconSize;
                     })
-                    .Foreground((Brush)Microsoft.UI.Xaml.Application.Current.Resources["TextOnAccentFillColorPrimaryBrush"]),
+                    .Foreground((Brush)Microsoft.UI.Xaml.Application.Current.Resources["SolidBackgroundFillColorBaseBrush"]),
                 Props.OnStop
             ).Set(b =>
             {
                 b.Padding = new Thickness(0);
                 b.MinWidth = 40; b.MinHeight = 32; b.Height = 32;
                 b.CornerRadius = controlCornerRadius;
-                b.Background = sendBrush;
+                b.Background = (Brush)Microsoft.UI.Xaml.Application.Current.Resources["TextFillColorPrimaryBrush"];
             })
             .Resources(r =>
             {
-                r.Set("ButtonBackgroundPointerOver", Ref("AccentFillColorSecondaryBrush"));
-                r.Set("ButtonBackgroundPressed", Ref("AccentFillColorTertiaryBrush"));
+                r.Set("ButtonBackgroundPointerOver", Ref("TextFillColorSecondaryBrush"));
+                r.Set("ButtonBackgroundPressed", Ref("TextFillColorTertiaryBrush"));
                 r.Set("ButtonBorderBrush", new SolidColorBrush(Colors.Transparent));
                 r.Set("ButtonBorderBrushPointerOver", new SolidColorBrush(Colors.Transparent));
                 r.Set("ButtonBorderBrushPressed", new SolidColorBrush(Colors.Transparent));

tests/OpenClaw.Tray.UITests/SessionTitleBehaviorProofTests.cs

@@ -122,9 +122,13 @@ private string WaitForSelectedSession(string expectedRouteTitle)
         var composerCondition = new PropertyCondition(
             AutomationElement.AutomationIdProperty,
             "ChatComposerInput");
-        var sessionSelectorCondition = new AndCondition(
-            new PropertyCondition(AutomationElement.ControlTypeProperty, ControlType.ComboBox),
-            new PropertyCondition(AutomationElement.NameProperty, "Session"));
+        // The redesigned session selector is a subtle menu-flyout Button (not a
+        // ComboBox). Its accessible name folds in the current selection as
+        // "Session: <route title>", replacing the legacy ComboBox's
+        // SelectionPattern. Match on the route title so the exact field-label
+        // prefix/separator format stays free to change.
+        var buttonCondition = new PropertyCondition(
+            AutomationElement.ControlTypeProperty, ControlType.Button);
 
         string? selectedRouteTitle = null;
         WaitUntil(() =>
@@ -133,21 +137,19 @@ private string WaitForSelectedSession(string expectedRouteTitle)
             if (hub.FindFirst(TreeScope.Descendants, composerCondition) is null)
                 return false;
 
-            var selector = hub.FindFirst(TreeScope.Descendants, sessionSelectorCondition);
-            if (selector is null
-                || !selector.TryGetCurrentPattern(SelectionPattern.Pattern, out var pattern)
-                || pattern is not SelectionPattern selection)
+            var buttons = hub.FindAll(TreeScope.Descendants, buttonCondition);
+            for (var i = 0; i < buttons.Count; i++)
             {
-                return false;
+                var name = buttons[i].Current.Name;
+                if (!string.IsNullOrEmpty(name)
+                    && name.Contains(expectedRouteTitle, StringComparison.Ordinal))
+                {
+                    selectedRouteTitle = name;
+                    return true;
+                }
             }
 
-            selectedRouteTitle = selection.Current.GetSelection()
-                .Select(item => item.Current.Name)
-                .SingleOrDefault();
-            return string.Equals(
-                selectedRouteTitle,
-                expectedRouteTitle,
-                StringComparison.Ordinal);
+            return false;
         }, $"chat Session selector to choose '{expectedRouteTitle}'");
 
         return Assert.IsType<string>(selectedRouteTitle);