From a5e6ce87e7c3d3443388693f736cbceb2da92316 Mon Sep 17 00:00:00 2001
From: DrSmugleaf <DrSmugleaf@users.noreply.github.com>
Date: Sun, 28 Nov 2021 14:19:44 +0100
Subject: [PATCH] Add a select all players button to the logs UI

---
 .../UI/CustomControls/AdminLogPlayerButton.cs |  1 -
 .../UI/CustomControls/AdminLogTypeButton.cs   |  1 -
 .../UI/Logs/AdminLogsWindow.xaml              |  8 +++--
 .../UI/Logs/AdminLogsWindow.xaml.cs           | 36 +++++++++++++++----
 4 files changed, 35 insertions(+), 11 deletions(-)

diff --git a/Content.Client/Administration/UI/CustomControls/AdminLogPlayerButton.cs b/Content.Client/Administration/UI/CustomControls/AdminLogPlayerButton.cs
index a6606a066a..3e4c71732b 100644
--- a/Content.Client/Administration/UI/CustomControls/AdminLogPlayerButton.cs
+++ b/Content.Client/Administration/UI/CustomControls/AdminLogPlayerButton.cs
@@ -10,7 +10,6 @@ public class AdminLogPlayerButton : Button
         Id = id;
         ClipText = true;
         ToggleMode = true;
-        Pressed = true;
     }
 
     public Guid Id { get; }
diff --git a/Content.Client/Administration/UI/CustomControls/AdminLogTypeButton.cs b/Content.Client/Administration/UI/CustomControls/AdminLogTypeButton.cs
index 8b93dfdec4..0f28a61fd4 100644
--- a/Content.Client/Administration/UI/CustomControls/AdminLogTypeButton.cs
+++ b/Content.Client/Administration/UI/CustomControls/AdminLogTypeButton.cs
@@ -10,7 +10,6 @@ public class AdminLogTypeButton : Button
         Type = type;
         ClipText = true;
         ToggleMode = true;
-        Pressed = true;
     }
 
     public LogType Type { get; }
diff --git a/Content.Client/Administration/UI/Logs/AdminLogsWindow.xaml b/Content.Client/Administration/UI/Logs/AdminLogsWindow.xaml
index 345e30cd27..1b47d0ee02 100644
--- a/Content.Client/Administration/UI/Logs/AdminLogsWindow.xaml
+++ b/Content.Client/Administration/UI/Logs/AdminLogsWindow.xaml
@@ -30,8 +30,12 @@
                 <BoxContainer Orientation="Vertical" MinWidth="200">
                     <LineEdit Name="PlayerSearch" Access="Public" StyleClasses="actionSearchBox"
                               HorizontalExpand="true" PlaceHolder="{Loc admin-logs-search-players-placeholder}"/>
-                    <Button Name="SelectNoPlayersButton" Text="{Loc admin-logs-select-none}"
-                            StyleClasses="ButtonSquare"/>
+                    <BoxContainer Orientation="Horizontal">
+                        <Button Name="SelectAllPlayersButton" Text="{Loc admin-logs-select-all}"
+                                MinWidth="100" StyleClasses="ButtonSquare"></Button>
+                        <Button Name="SelectNoPlayersButton" Text="{Loc admin-logs-select-none}"
+                                MinWidth="100" StyleClasses="ButtonSquare"/>
+                    </BoxContainer>
                     <ScrollContainer VerticalExpand="True">
                         <BoxContainer Name="PlayersContainer" Access="Public" Orientation="Vertical"/>
                     </ScrollContainer>
diff --git a/Content.Client/Administration/UI/Logs/AdminLogsWindow.xaml.cs b/Content.Client/Administration/UI/Logs/AdminLogsWindow.xaml.cs
index ddf1549fa7..ac1679ac71 100644
--- a/Content.Client/Administration/UI/Logs/AdminLogsWindow.xaml.cs
+++ b/Content.Client/Administration/UI/Logs/AdminLogsWindow.xaml.cs
@@ -35,6 +35,7 @@ public partial class AdminLogsWindow : SS14Window
         SelectAllTypesButton.OnPressed += SelectAllTypes;
         SelectNoTypesButton.OnPressed += SelectNoTypes;
 
+        SelectAllPlayersButton.OnPressed += SelectAllPlayers;
         SelectNoPlayersButton.OnPressed += SelectNoPlayers;
 
         RoundSpinBox.IsValid = i => i > 0 && i <= CurrentRound;
@@ -90,7 +91,7 @@ public partial class AdminLogsWindow : SS14Window
         UpdateTypes();
     }
 
-    private void PlayerSearchChanged(LineEditEventArgs obj)
+    private void PlayerSearchChanged(LineEditEventArgs args)
     {
         UpdatePlayers();
     }
@@ -100,7 +101,7 @@ public partial class AdminLogsWindow : SS14Window
         UpdateLogs();
     }
 
-    private void SelectAllTypes(ButtonEventArgs obj)
+    private void SelectAllTypes(ButtonEventArgs args)
     {
         SelectedTypes.Clear();
 
@@ -118,7 +119,7 @@ public partial class AdminLogsWindow : SS14Window
         UpdateLogs();
     }
 
-    private void SelectNoTypes(ButtonEventArgs obj)
+    private void SelectNoTypes(ButtonEventArgs args)
     {
         SelectedTypes.Clear();
 
@@ -136,7 +137,25 @@ public partial class AdminLogsWindow : SS14Window
         UpdateLogs();
     }
 
-    private void SelectNoPlayers(ButtonEventArgs obj)
+    private void SelectAllPlayers(ButtonEventArgs args)
+    {
+        SelectedPlayers.Clear();
+
+        foreach (var control in PlayersContainer.Children)
+        {
+            if (control is not AdminLogPlayerButton player)
+            {
+                continue;
+            }
+
+            player.Pressed = true;
+            SelectedPlayers.Add(player.Id);
+        }
+
+        UpdateLogs();
+    }
+
+    private void SelectNoPlayers(ButtonEventArgs args)
     {
         SelectedPlayers.Clear();
 
@@ -311,7 +330,8 @@ public partial class AdminLogsWindow : SS14Window
         {
             var button = new AdminLogTypeButton(type)
             {
-                Text = type.ToString()
+                Text = type.ToString(),
+                Pressed = true
             };
 
             SelectedTypes.Add(type);
@@ -349,7 +369,8 @@ public partial class AdminLogsWindow : SS14Window
         {
             var button = new AdminLogPlayerButton(id)
             {
-                Text = name
+                Text = name,
+                Pressed = true
             };
 
             SelectedPlayers.Add(id);
@@ -397,8 +418,9 @@ public partial class AdminLogsWindow : SS14Window
         LogSearch.OnTextChanged -= LogSearchChanged;
 
         SelectAllTypesButton.OnPressed -= SelectAllTypes;
-
         SelectNoTypesButton.OnPressed -= SelectNoTypes;
+
+        SelectAllPlayersButton.OnPressed -= SelectAllPlayers;
         SelectNoPlayersButton.OnPressed -= SelectNoPlayers;
 
         RoundSpinBox.IsValid = null;