From f7259705328f6dfc24ad13697b7ea3d40976a828 Mon Sep 17 00:00:00 2001
From: rampaa <rampaa@waifuland.com>
Date: Sun, 6 Oct 2024 20:38:46 +0300
Subject: [PATCH] Refactor the theme selection code and add a new theme

---
 JL.Windows/ConfigManager.cs           | 23 +++++++----------------
 JL.Windows/GUI/PreferencesWindow.xaml |  3 ++-
 2 files changed, 9 insertions(+), 17 deletions(-)

diff --git a/JL.Windows/ConfigManager.cs b/JL.Windows/ConfigManager.cs
index 2ce824a8..e0fdd53a 100644
--- a/JL.Windows/ConfigManager.cs
+++ b/JL.Windows/ConfigManager.cs
@@ -183,27 +183,18 @@ internal static class ConfigManager
 
     private static readonly ComboBoxItem[] s_japaneseFonts = WindowsUtils.FindJapaneseFonts();
     private static readonly ComboBoxItem[] s_popupJapaneseFonts = WindowsUtils.CloneJapaneseFontComboBoxItems(s_japaneseFonts);
-    private static SkinType s_theme = SkinType.Dark;
+    private static SkinType Theme { get; set; } = SkinType.Dark;
 
     public static void ApplyPreferences()
     {
         using SqliteConnection connection = ConfigDBManager.CreateReadWriteDBConnection();
         CoreConfigManager.ApplyPreferences(connection);
 
+        SkinType skinType = ConfigDBManager.GetValueFromConfig(connection, Theme, nameof(Theme), Enum.TryParse);
+        if (skinType != Theme)
         {
-            string? themeStr = ConfigDBManager.GetSettingValue(connection, "Theme");
-            if (themeStr is null)
-            {
-                themeStr = nameof(SkinType.Dark);
-                ConfigDBManager.InsertSetting(connection, "Theme", themeStr);
-            }
-
-            SkinType skinType = themeStr is nameof(SkinType.Dark) ? SkinType.Dark : SkinType.Default;
-            if (s_theme != skinType)
-            {
-                s_theme = skinType;
-                WindowsUtils.ChangeTheme(s_theme);
-            }
+            Theme = skinType;
+            WindowsUtils.ChangeTheme(Theme);
         }
 
         if (CoreConfigManager.CaptureTextFromClipboard)
@@ -950,7 +941,7 @@ public static void LoadPreferenceWindow(PreferencesWindow preferenceWindow)
         using SqliteConnection connection = ConfigDBManager.CreateReadOnlyDBConnection();
         preferenceWindow.ProfileComboBox.ItemsSource = ProfileDBUtils.GetProfileNames(connection);
         preferenceWindow.ProfileComboBox.SelectedItem = ProfileUtils.CurrentProfileName;
-        preferenceWindow.ThemeComboBox.SelectedValue = ConfigDBManager.GetSettingValue(connection, "Theme");
+        preferenceWindow.ThemeComboBox.SelectedValue = ConfigDBManager.GetSettingValue(connection, nameof(Theme));
         preferenceWindow.MainWindowTextVerticalAlignmentComboBox.SelectedValue = ConfigDBManager.GetSettingValue(connection, nameof(MainWindowTextVerticalAlignment));
         preferenceWindow.MinimumLogLevelComboBox.SelectedValue = ConfigDBManager.GetSettingValue(connection, "MinimumLogLevel");
         preferenceWindow.PopupPositionRelativeToCursorComboBox.SelectedValue = ConfigDBManager.GetSettingValue(connection, "PopupPositionRelativeToCursor");
@@ -1154,7 +1145,7 @@ public static async Task SavePreferences(PreferencesWindow preferenceWindow)
             ConfigDBManager.UpdateSetting(connection, "MainWindowOpacity",
                 preferenceWindow.MainWindowOpacityNumericUpDown.Value.ToString(CultureInfo.InvariantCulture));
 
-            ConfigDBManager.UpdateSetting(connection, "Theme", preferenceWindow.ThemeComboBox.SelectedValue.ToString()!);
+            ConfigDBManager.UpdateSetting(connection, nameof(Theme), preferenceWindow.ThemeComboBox.SelectedValue.ToString()!);
             ConfigDBManager.UpdateSetting(connection, nameof(MainWindowTextVerticalAlignment), preferenceWindow.MainWindowTextVerticalAlignmentComboBox.SelectedValue.ToString()!);
 
             ConfigDBManager.UpdateSetting(connection, "MinimumLogLevel", preferenceWindow.MinimumLogLevelComboBox.SelectedValue.ToString()!);
diff --git a/JL.Windows/GUI/PreferencesWindow.xaml b/JL.Windows/GUI/PreferencesWindow.xaml
index 57429f12..52771ecb 100644
--- a/JL.Windows/GUI/PreferencesWindow.xaml
+++ b/JL.Windows/GUI/PreferencesWindow.xaml
@@ -30,7 +30,8 @@
                             <ComboBox x:Name="ThemeComboBox" HorizontalAlignment="Right"
                                       SelectedValuePath="Tag" Width="250">
                                 <ComboBoxItem Content="Dark" Tag="Dark" />
-                                <ComboBoxItem Content="Light" Tag="Light" />
+                                <ComboBoxItem Content="Light" Tag="Default" />
+                                <ComboBoxItem Content="Violet" Tag="Violet" />
                             </ComboBox>
                         </DockPanel>