From 9fd01c8f9f6fb109bbb8f255db4378d573040201 Mon Sep 17 00:00:00 2001 From: legoandmars Date: Wed, 14 Feb 2024 19:43:28 -0800 Subject: [PATCH] Move default color values to constant --- LobbyCompatibility/Configuration/Config.cs | 21 ++++++++++++++++++--- LobbyCompatibility/Models/PluginDiff.cs | 7 ++++--- 2 files changed, 22 insertions(+), 6 deletions(-) diff --git a/LobbyCompatibility/Configuration/Config.cs b/LobbyCompatibility/Configuration/Config.cs index 69a3195..1015481 100644 --- a/LobbyCompatibility/Configuration/Config.cs +++ b/LobbyCompatibility/Configuration/Config.cs @@ -9,6 +9,21 @@ namespace LobbyCompatibility.Configuration; /// public class Config { + /// + /// Default used to represent compatible plugins. + /// + public static readonly Color DefaultCompatibleColor = Color.green; + + /// + /// Default used to represent incompatible plugins. + /// + public static readonly Color DefaultIncompatibleColor = Color.red; + + /// + /// Default used to represent unknown plugins. + /// + public static readonly Color DefaultUnknownColor = Color.gray; + /// /// Default value for public lobby sorting. /// @@ -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 index 533765b..41ca6e7 100644 --- 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 }; } } \ No newline at end of file