Move default color values to constant

MaxWasUnavailable · Feb 15, 2024 · 9fd01c8 · 9fd01c8
1 parent 9c4e23e
commit 9fd01c8
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 6 deletions.
diff --git a/LobbyCompatibility/Configuration/Config.cs b/LobbyCompatibility/Configuration/Config.cs
@@ -9,6 +9,21 @@ namespace LobbyCompatibility.Configuration;
 /// </summary>
 public class Config
 {
+    /// <summary>
+    ///     Default <see cref="Color"/> used to represent compatible plugins.
+    /// </summary>
+    public static readonly Color DefaultCompatibleColor = Color.green;
+
+    /// <summary>
+    ///     Default <see cref="Color"/> used to represent incompatible plugins.
+    /// </summary>
+    public static readonly Color DefaultIncompatibleColor = Color.red;
+
+    /// <summary>
+    ///     Default <see cref="Color"/> used to represent unknown plugins.
+    /// </summary>
+    public static readonly Color DefaultUnknownColor = Color.gray;
+
     /// <summary>
     ///     Default <see cref="ModdedLobbyFilter"/> value for public lobby sorting.
     /// </summary>
@@ -46,15 +61,15 @@ public Config(ConfigFile configFile)
             "The default tab to use when viewing a lobby's mod list");
         CompatibleColor = configFile.Bind("Visual",
             "Compatible Plugin Color",
-            Color.green,
+            DefaultCompatibleColor,
             "The color used to respresent compatible plugins");
         IncompatibleColor = configFile.Bind("Visual",
             "Incompatible Plugin Color",
-            Color.red,
+            DefaultIncompatibleColor,
             "The color used to respresent incompatible plugins");
         UnknownColor = configFile.Bind("Visual",
             "Unknown Plugin Color",
-            Color.gray,
+            DefaultUnknownColor,
             "The color used to respresent unknown plugins");
     }
 }
diff --git a/LobbyCompatibility/Models/PluginDiff.cs b/LobbyCompatibility/Models/PluginDiff.cs
@@ -1,4 +1,5 @@
 using System;
+using LobbyCompatibility.Configuration;
 using LobbyCompatibility.Enums;
 using UnityEngine;
 
@@ -42,10 +43,10 @@ public Color GetTextColor()
     {
         return PluginDiffResult switch
         {
-            PluginDiffResult.Compatible => LobbyCompatibilityPlugin.Config?.CompatibleColor.Value ?? Color.green,
+            PluginDiffResult.Compatible => LobbyCompatibilityPlugin.Config?.CompatibleColor.Value ?? Config.DefaultCompatibleColor,
             PluginDiffResult.ClientMissingMod or PluginDiffResult.ServerMissingMod or PluginDiffResult.ModVersionMismatch 
-                => LobbyCompatibilityPlugin.Config?.IncompatibleColor.Value ?? Color.red,
-            _ => LobbyCompatibilityPlugin.Config?.UnknownColor.Value ?? Color.gray
+                => LobbyCompatibilityPlugin.Config?.IncompatibleColor.Value ?? Config.DefaultIncompatibleColor,
+            _ => LobbyCompatibilityPlugin.Config?.UnknownColor.Value ?? Config.DefaultUnknownColor
         };
     }
 }