From 3eb28c57f225445f7f5ef8b4e79d5015f513d9e9 Mon Sep 17 00:00:00 2001 From: Erik Date: Fri, 21 Aug 2026 06:33:48 +0200 Subject: [PATCH] fix(chat): the talk-focus button names the focus, not the target MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 581a61ef made the chat button display the tell TARGET's name once "Tell to X" was picked. It should read "Tell" — the button names the focus, the same way it reads "Chat", "General" or "Fellow" for the other focuses. Reported against retail. Co-Authored-By: Claude Opus 5 --- src/AcDream.App/UI/Layout/ChatWindowController.cs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/AcDream.App/UI/Layout/ChatWindowController.cs b/src/AcDream.App/UI/Layout/ChatWindowController.cs index e1392ee0..da8e21df 100644 --- a/src/AcDream.App/UI/Layout/ChatWindowController.cs +++ b/src/AcDream.App/UI/Layout/ChatWindowController.cs @@ -173,6 +173,7 @@ public sealed class ChatWindowController : IRetainedWindowStateController, IReta private static string ChannelButtonLabel(ChatChannelKind k) => k switch { ChatChannelKind.Say => "Chat", + ChatChannelKind.Tell => "Tell", ChatChannelKind.General => "General", ChatChannelKind.Trade => "Trade", ChatChannelKind.Lfg => "LFG", @@ -428,10 +429,9 @@ public sealed class ChatWindowController : IRetainedWindowStateController, IReta TalkFocusSpecial => SelectedName() is not null, _ => true, }; - menu.ButtonLabelProvider = () => - c._activeChannel == ChatChannelKind.Tell && c._tellTarget is { } t - ? t - : ChannelButtonLabel(c._activeChannel); + // The button names the FOCUS, not the target: retail shows "Tell", + // not the person's name. + menu.ButtonLabelProvider = () => ChannelButtonLabel(c._activeChannel); menu.OnOpen = RebuildItems; menu.OnSelect = p => {