Refactor Application Settings and Support for Packaged/Unpackaged Mode #1924

WinUIGallery/App.xaml.cs

@@ -58,6 +58,13 @@ protected override void OnLaunched(Microsoft.UI.Xaml.LaunchActivatedEventArgs ar
 
         EnsureWindow();
 
+        if (SettingsHelper.Current.IsFirstRun)
+        {
+            SettingsMigration.MigrateRecentlyVisited();
+            SettingsMigration.MigrateFavorites();
+            SettingsHelper.Current.IsFirstRun = false;
+        }
+
         MainWindow.Closed += (s, e) =>
         {
             if (IsAppPackaged)

WinUIGallery/Controls/PageHeader.xaml.cs

@@ -101,7 +101,7 @@ private void UserControl_Loaded(object sender, RoutedEventArgs e)
         }
         if (Item != null)
         {
-            FavoriteButton.IsChecked = SettingsHelper.Contains(SettingsKeys.Favorites, Item.UniqueId);
+            FavoriteButton.IsChecked = SettingsHelper.Current.Favorites.Contains(Item.UniqueId);
         }
     }
 
@@ -121,11 +121,11 @@ private void FavoriteButton_Click(object sender, RoutedEventArgs e)
         {
             if (toggleButton.IsChecked == true)
             {
-                SettingsHelper.TryAddItem(SettingsKeys.Favorites, Item.UniqueId, InsertPosition.Last);
+                SettingsHelper.Current.UpdateFavorites(items => items.AddAsLast(Item.UniqueId));
             }
             else
             {
-                SettingsHelper.TryRemoveItem(SettingsKeys.Favorites, Item.UniqueId);
+                SettingsHelper.Current.UpdateFavorites(items => items.Remove(Item.UniqueId));
             }
         }
     }

WinUIGallery/Helpers/NavigationOrientationHelper.cs

@@ -2,43 +2,20 @@
 // Licensed under the MIT License.
 
 using Microsoft.UI.Xaml.Controls;
-using Microsoft.Windows.Storage;
 
 namespace WinUIGallery.Helpers;
 
 public static partial class NavigationOrientationHelper
 {
-    private static bool _isLeftMode = true;
-    private static ApplicationData appData = ApplicationData.GetDefault();
     public static bool IsLeftMode()
     {
-        if (NativeMethods.IsAppPackaged)
-        {
-            var valueFromSettings = appData.LocalSettings.Values[SettingsKeys.IsLeftMode];
-            if (valueFromSettings == null)
-            {
-                appData.LocalSettings.Values[SettingsKeys.IsLeftMode] = true;
-                valueFromSettings = true;
-            }
-            return (bool)valueFromSettings;
-        }
-        else
-        {
-            return _isLeftMode;
-        }
+        return SettingsHelper.Current.IsLeftMode;
     }
 
     public static void IsLeftModeForElement(bool isLeftMode)
     {
         UpdateNavigationViewForElement(isLeftMode);
-        if (NativeMethods.IsAppPackaged)
-        {
-            appData.LocalSettings.Values[SettingsKeys.IsLeftMode] = isLeftMode;
-        }
-        else
-        {
-            _isLeftMode = isLeftMode;
-        }
+        SettingsHelper.Current.IsLeftMode = isLeftMode;
     }
 
     public static void UpdateNavigationViewForElement(bool isLeftMode)

WinUIGallery/Helpers/ProtocolActivationClipboardHelper.cs

@@ -15,40 +15,16 @@ namespace WinUIGallery.Helpers;
 /// </summary>
 public static partial class ProtocolActivationClipboardHelper
 {
-    private static bool _showCopyLinkTeachingTip = true;
-    private static ApplicationData appData = ApplicationData.GetDefault();
-
     public static bool ShowCopyLinkTeachingTip
     {
         get
         {
-            if (NativeMethods.IsAppPackaged)
-            {
-                object valueFromSettings = appData.LocalSettings.Values[SettingsKeys.ShowCopyLinkTeachingTip];
-                if (valueFromSettings == null)
-                {
-                    appData.LocalSettings.Values[SettingsKeys.ShowCopyLinkTeachingTip] = true;
-                    valueFromSettings = true;
-                }
-                return (bool)valueFromSettings;
-            }
-            else
-            {
-                return _showCopyLinkTeachingTip;
-            }
+            return SettingsHelper.Current.IsShowCopyLinkTeachingTip;
         }
 
         set
         {
-            if (NativeMethods.IsAppPackaged)
-            {
-                appData.LocalSettings.Values[SettingsKeys.ShowCopyLinkTeachingTip] = value;
-
-            }
-            else
-            {
-                _showCopyLinkTeachingTip = value;
-            }
+            SettingsHelper.Current.IsShowCopyLinkTeachingTip = value;
         }
     }
 

WinUIGallery/Helpers/SettingsHelper/Internals/ListExtensions.cs

@@ -0,0 +1,34 @@
+using System.Collections.Generic;
+
+namespace WinUIGallery.Helpers;
+
+internal static class ListExtensions
+{
+    public static void AddAsFirst<T>(this List<T> list, T item, int? maxSize = null)
+    {
+        if (item == null || (item is string data && string.IsNullOrWhiteSpace(data)))
+            return;
+
+        list.Remove(item);
+        list.Insert(0, item);
+
+        if (maxSize.HasValue && maxSize.Value > 0 && list.Count > maxSize.Value)
+        {
+            list.RemoveRange(maxSize.Value, list.Count - maxSize.Value);
+        }
+    }
+
+    public static void AddAsLast<T>(this List<T> list, T item, int? maxSize = null)
+    {
+        if (item == null || (item is string data && string.IsNullOrWhiteSpace(data)))
+            return;
+
+        list.Remove(item);
+        list.Add(item);
+
+        if (maxSize.HasValue && maxSize.Value > 0 && list.Count > maxSize.Value)
+        {
+            list.RemoveRange(0, list.Count - maxSize.Value);
+        }
+    }
+}

WinUIGallery/Helpers/SettingsHelper/Internals/SettingsJsonContext.cs

@@ -0,0 +1,11 @@
+using System.Collections.Generic;
+using System.Text.Json.Serialization;
+
+namespace WinUIGallery.Helpers;
+
+[JsonSourceGenerationOptions(WriteIndented = true, GenerationMode = JsonSourceGenerationMode.Metadata)]
+[JsonSerializable(typeof(Dictionary<string, System.Text.Json.JsonElement>))]
+[JsonSerializable(typeof(SettingsHelper))]
+internal partial class SettingsJsonContext : JsonSerializerContext
+{
+}