tommy/tbd-station-14
2
1
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix PlayerListControl re-raising selection changed when repopulated. (#25412)

Browse Source 
Now we just skip duplicate "item pressed" events from the ListContainer.

This caused the ahelp window to unfocus the message box after sending something. Flow is something like this: you send ahelp -> bwoink window refreshes player list due to new bwoink -> repopulated player list -> sent selection change -> repopulates right pane -> line edit gets unfocused.
This commit is contained in:
Pieter-Jan Briers
2024-02-20 14:01:28 +01:00
committed by GitHub
parent bd40c0ce26
commit 84d455b9f8
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
Content.Client/Administration/UI/CustomControls/PlayerListControl.xaml.cs
View File

@@ -52,6 +52,9 @@ namespace Content.Client.Administration.UI.CustomControls
if (args == null || data is not PlayerListData {Info: var selectedPlayer})
return;
if (selectedPlayer == _selectedPlayer)
return;
if (args.Event.Function != EngineKeyFunctions.UIClick)
return;