gui-cs
diff --git a/‎Examples/UICatalog/Scenarios/CombiningMarks.cs‎
Lines changed: 1 addition & 1 deletion b/‎Examples/UICatalog/Scenarios/CombiningMarks.cs‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎Examples/UICatalog/Scenarios/Shortcuts.cs‎
Lines changed: 2 additions & 0 deletions b/‎Examples/UICatalog/Scenarios/Shortcuts.cs‎
Lines changed: 2 additions & 0 deletions
diff --git a/‎Examples/UICatalog/Scenarios/Themes.cs‎
Lines changed: 1 addition & 1 deletion b/‎Examples/UICatalog/Scenarios/Themes.cs‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎Terminal.Gui/App/Application.Screen.cs‎
Lines changed: 2 additions & 2 deletions b/‎Terminal.Gui/App/Application.Screen.cs‎
Lines changed: 2 additions & 2 deletions
diff --git a/‎Terminal.Gui/Configuration/ConfigProperty.cs‎
Lines changed: 3 additions & 0 deletions b/‎Terminal.Gui/Configuration/ConfigProperty.cs‎
Lines changed: 3 additions & 0 deletions
diff --git a/‎Terminal.Gui/Configuration/ConfigurationManager.cs‎
Lines changed: 30 additions & 20 deletions b/‎Terminal.Gui/Configuration/ConfigurationManager.cs‎
Lines changed: 30 additions & 20 deletions
diff --git a/‎Terminal.Gui/Configuration/SchemeManager.cs‎
Lines changed: 25 additions & 15 deletions b/‎Terminal.Gui/Configuration/SchemeManager.cs‎
Lines changed: 25 additions & 15 deletions
@@ -13,7 +13,7 @@ public override void Main ()
         top.DrawComplete += (s, e) =>
         {
             // Forces reset _lineColsOffset because we're dealing with direct draw
-            Application.ClearScreenNextIteration = true;
+            Application.Top!.SetNeedsDraw ();
 
             var i = -1;
             top.AddStr ("Terminal.Gui only supports combining marks that normalize. See Issue #2616.");
 
@@ -12,13 +12,15 @@ public class Shortcuts : Scenario
     public override void Main ()
     {
         Application.Init ();
+        var quitKey = Application.QuitKey;
         Window app = new ();
 
         app.Loaded += App_Loaded;
 
         Application.Run (app);
         app.Dispose ();
         Application.Shutdown ();
+        Application.QuitKey = quitKey;
     }
 
     // Setting everything up in Loaded handler because we change the
 
@@ -172,7 +172,7 @@ public override void Main ()
                                                     else
                                                     {
                                                         appWindow.Remove (allViewsView);
-                                                        allViewsView.Dispose ();
+                                                        allViewsView!.Dispose ();
                                                         allViewsView = null;
 
                                                         appWindow.Add (viewFrame);
 
@@ -82,8 +82,8 @@ public static bool OnSizeChanging (SizeChangedEventArgs args)
     ///     Gets or sets whether the screen will be cleared, and all Views redrawn, during the next Application iteration.
     /// </summary>
     /// <remarks>
-    ///     This is typicall set to true when a View's <see cref="View.Frame"/> changes and that view has no
+    ///     This is typical set to true when a View's <see cref="View.Frame"/> changes and that view has no
     ///     SuperView (e.g. when <see cref="Application.Top"/> is moved or resized.
     /// </remarks>
-    public static bool ClearScreenNextIteration { get; set; }
+    internal static bool ClearScreenNextIteration { get; set; }
 }
@@ -1,6 +1,7 @@
 #nullable enable
 using System.Collections.Concurrent;
 using System.Collections.Immutable;
+using System.Diagnostics;
 using System.Diagnostics.CodeAnalysis;
 using System.Reflection;
 using System.Text.Json;
@@ -74,6 +75,8 @@ public bool Apply ()
             {
                 // Use DeepCloner to create a deep copy of PropertyValue
                 object? val = DeepCloner.DeepClone (PropertyValue);
+
+                Debug.Assert (!Immutable);
                 PropertyInfo.SetValue (null, val);
 
             }
 
@@ -51,7 +51,7 @@ public static class ConfigurationManager
 {
     /// <summary>The backing property for <see cref="Settings"/> (config settings of <see cref="SettingsScope"/>).</summary>
     /// <remarks>
-    ///     Is <see langword="null"/> until <see cref="ResetToCurrentValues"/> is called. Gets set to a new instance by
+    ///     Is <see langword="null"/> until <see cref="UpdateToCurrentValues"/> is called. Gets set to a new instance by
     ///     deserialization
     ///     (see <see cref="Load"/>).
     /// </remarks>
@@ -117,15 +117,19 @@ internal static bool IsInitialized ()
         }
     }
 
+    // TODO: Find a way to make this cache truly read-only at the leaf node level. 
+    // TODO: Right now, the dictionary is frozen, but the ConfigProperty instances can still be modified   
+    // TODO: if the PropertyValue is a reference type.
+    // TODO: See https://github.com/gui-cs/Terminal.Gui/issues/4288
     /// <summary>
     ///     A cache of all<see cref="ConfigurationPropertyAttribute"/> properties and their hard coded values.
     /// </summary>
     /// <remarks>Is <see langword="null"/> until <see cref="Initialize"/> is called.</remarks>
 #pragma warning disable IDE1006 // Naming Styles
     internal static FrozenDictionary<string, ConfigProperty>? _hardCodedConfigPropertyCache;
+
     private static readonly object _hardCodedConfigPropertyCacheLock = new ();
 #pragma warning restore IDE1006 // Naming Styles
-
     internal static FrozenDictionary<string, ConfigProperty>? GetHardCodedConfigPropertyCache ()
     {
         lock (_hardCodedConfigPropertyCacheLock)
@@ -183,7 +187,7 @@ internal static void Initialize ()
         lock (_uninitializedConfigPropertiesCacheCacheLock)
         {
             // _allConfigProperties: for ordered, iterable access (LINQ-friendly)
-            // _frozenConfigPropertyCache: for high-speed key lookup (frozen)
+            // _hardCodedConfigPropertyCache: for high-speed key lookup (frozen)
 
             // Note GetAllConfigProperties returns a new instance and all the properties !HasValue and Immutable.
             _uninitializedConfigPropertiesCache = ConfigProperty.GetAllConfigProperties ();
@@ -209,6 +213,11 @@ internal static void Initialize ()
         }
 
         LoadHardCodedDefaults ();
+
+        // BUGBUG: ThemeScope is broken and needs to be fixed to not have the hard coded schemes get overwritten.
+        // BUGBUG: This a partial workaround.
+        // BUGBUG: See https://github.com/gui-cs/Terminal.Gui/issues/4288
+        ThemeManager.Themes? [ThemeManager.Theme]?.Apply ();
     }
 
     #endregion Initialization
@@ -291,6 +300,7 @@ public static void Disable (bool resetToHardCodedDefaults = false)
 
         if (resetToHardCodedDefaults)
         {
+            // Calls Apply
             ResetToHardCodedDefaults ();
         }
     }
@@ -299,16 +309,17 @@ public static void Disable (bool resetToHardCodedDefaults = false)
 
     #region Reset
 
-    // `Reset` - Reset the configuration to either the current values or the hard-coded defaults.
-    // Resetting does not load the configuration; it only resets the configuration to the default values.
+    // `Update` - Updates the configuration from either the current values or the hard-coded defaults.
+    // Updating does not load the configuration; it only updates the configuration to the values currently
+    // in the static ConfigProperties.
 
     /// <summary>
-    ///     INTERNAL: Resets <see cref="ConfigurationManager"/>. Loads settings from the current
+    ///     INTERNAL: Updates <see cref="ConfigurationManager"/> to the settings from the current
     ///     values of the static <see cref="ConfigurationPropertyAttribute"/> properties.
     /// </summary>
     [RequiresUnreferencedCode ("AOT")]
     [RequiresDynamicCode ("AOT")]
-    internal static void ResetToCurrentValues ()
+    internal static void UpdateToCurrentValues ()
     {
         if (!IsInitialized ())
         {
@@ -327,13 +338,13 @@ internal static void ResetToCurrentValues ()
             _settingsLockSlim.ExitWriteLock ();
         }
 
-        Settings!.LoadCurrentValues ();
+        Settings!.UpdateToCurrentValues ();
         ThemeManager.UpdateToCurrentValues ();
-        AppSettings!.LoadCurrentValues ();
+        AppSettings!.UpdateToCurrentValues ();
     }
 
     /// <summary>
-    ///     INTERNAL: Resets <see cref="ConfigurationManager"/>. Loads the hard-coded values of the
+    ///     INTERNAL: Loads the hard-coded values of the
     ///     <see cref="ConfigurationPropertyAttribute"/> properties and applies them.
     /// </summary>
     [RequiresUnreferencedCode ("AOT")]
@@ -374,7 +385,7 @@ internal static void LoadHardCodedDefaults ()
 
         Settings = new ();
         Settings!.LoadHardCodedDefaults ();
-        ThemeManager.ResetToHardCodedDefaults ();
+        ThemeManager.LoadHardCodedDefaults ();
         AppSettings!.LoadHardCodedDefaults ();
     }
 
@@ -447,10 +458,6 @@ public static void Load (ConfigLocations locations)
         {
             SourcesManager?.Load (Settings, $"~/.tui/{AppName}.{_configFilename}", ConfigLocations.AppHome);
         }
-
-        Settings!.Validate ();
-        ThemeManager.Validate ();
-        AppSettings!.Validate ();
     }
 
     // TODO: Rename to Loaded?
@@ -566,7 +573,7 @@ private static void OnApplied ()
 
     [SuppressMessage ("Style", "IDE1006:Naming Styles", Justification = "<Pending>")]
     internal static readonly SourceGenerationContext SerializerContext = new (
-                                                                              new JsonSerializerOptions
+                                                                              new()
                                                                               {
                                                                                   // Be relaxed
                                                                                   ReadCommentHandling = JsonCommentHandling.Skip,
@@ -638,7 +645,7 @@ public static AppSettingsScope? AppSettings
                 if (!appSettingsConfigProperty.HasValue)
                 {
                     var appSettings = new AppSettingsScope ();
-                    appSettings.LoadCurrentValues ();
+                    appSettings.UpdateToCurrentValues ();
 
                     return appSettings;
                 }
@@ -710,8 +717,9 @@ public static void PrintJsonErrors ()
         {
             if (_jsonErrors.Length > 0)
             {
-                Console.WriteLine (@"Terminal.Gui ConfigurationManager encountered these errors while reading configuration files" +
-                                   @"(set ThrowOnJsonErrors to have these caught during execution):");
+                Console.WriteLine (
+                                   @"Terminal.Gui ConfigurationManager encountered these errors while reading configuration files"
+                                   + @"(set ThrowOnJsonErrors to have these caught during execution):");
                 Console.WriteLine (_jsonErrors.ToString ());
             }
         }
@@ -783,8 +791,10 @@ public static string GetHardCodedConfig ()
 
             Debug.Assert (filtered is { });
 
-            IEnumerable<KeyValuePair<string, ConfigProperty>> configPropertiesByScope = filtered as KeyValuePair<string, ConfigProperty> [] ?? filtered.ToArray ();
+            IEnumerable<KeyValuePair<string, ConfigProperty>> configPropertiesByScope =
+                filtered as KeyValuePair<string, ConfigProperty> [] ?? filtered.ToArray ();
             Debug.Assert (configPropertiesByScope.All (v => !v.Value.HasValue));
+
             return configPropertiesByScope;
         }
     }
 
@@ -7,25 +7,29 @@
 namespace Terminal.Gui.Configuration;
 
 /// <summary>
-///     Holds the <see cref="Drawing.Scheme"/>s that define the <see cref="System.Attribute"/>s that are used by views to render
-///     themselves. A Scheme is a mapping from <see cref="Drawing.VisualRole"/>s (such as <see cref="Drawing.VisualRole.Focus"/>) to <see cref="System.Attribute"/>s.
+///     Holds the <see cref="Drawing.Scheme"/>s that define the <see cref="System.Attribute"/>s that are used by views to
+///     render
+///     themselves. A Scheme is a mapping from <see cref="Drawing.VisualRole"/>s (such as
+///     <see cref="Drawing.VisualRole.Focus"/>) to <see cref="System.Attribute"/>s.
 ///     A Scheme defines how a `View` should look based on its purpose (e.g. Menu or Dialog).
 /// </summary>
-public sealed class SchemeManager// : INotifyCollectionChanged, IDictionary<string, Scheme?>
+public sealed class SchemeManager // : INotifyCollectionChanged, IDictionary<string, Scheme?>
 {
 #pragma warning disable IDE1006 // Naming Styles
     private static readonly object _schemesLock = new ();
 #pragma warning restore IDE1006 // Naming Styles
 
     /// <summary>
-    ///     INTERNAL: Gets the hard-coded schemes defined by <see cref="View"/>. These are not loaded from the configuration files,
+    ///     INTERNAL: Gets the hard-coded schemes defined by <see cref="View"/>. These are not loaded from the configuration
+    ///     files,
     ///     but are hard-coded in the source code. Used for unit testing when ConfigurationManager is not initialized.
     /// </summary>
     /// <returns></returns>
     internal static ImmutableSortedDictionary<string, Scheme?>? GetHardCodedSchemes () { return Scheme.GetHardCodedSchemes ()!; }
 
     /// <summary>
-    ///     Use <see cref="AddScheme"/>, <see cref="GetScheme(Drawing.Schemes)"/>, <see cref="GetSchemeNames"/>, <see cref="GetSchemesForCurrentTheme"/>, etc... instead.
+    ///     Use <see cref="AddScheme"/>, <see cref="GetScheme(Drawing.Schemes)"/>, <see cref="GetSchemeNames"/>,
+    ///     <see cref="GetSchemesForCurrentTheme"/>, etc... instead.
     /// </summary>
     [ConfigurationProperty (Scope = typeof (ThemeScope), OmitClassName = true)]
     [JsonConverter (typeof (DictionaryJsonConverter<Scheme?>))]
@@ -54,7 +58,7 @@ public sealed class SchemeManager// : INotifyCollectionChanged, IDictionary<stri
     /// <summary>INTERNAL: The set method for <see cref="Schemes"/>.</summary>
     [RequiresUnreferencedCode ("Calls Terminal.Gui.ConfigProperty.UpdateFrom(Object)")]
     [RequiresDynamicCode ("Calls Terminal.Gui.ConfigProperty.UpdateFrom(Object)")]
-    private static void SetSchemes (Dictionary<string, Scheme?>? value)
+    internal static void SetSchemes (Dictionary<string, Scheme?>? value)
     {
         lock (_schemesLock)
         {
@@ -117,6 +121,7 @@ public static Scheme GetScheme (Schemes schemeName)
     {
         // Convert schemeName to string via Enum api
         string? schemeNameString = SchemesToSchemeName (schemeName);
+
         if (schemeNameString is null)
         {
             throw new ArgumentException ($"Invalid scheme name: {schemeName}");
@@ -131,21 +136,15 @@ public static Scheme GetScheme (Schemes schemeName)
     /// <param name="schemeName"></param>
     /// <returns></returns>
     /// <exception cref="ArgumentException"></exception>
-    public static Scheme GetScheme (string schemeName)
-    {
-        return GetSchemesForCurrentTheme ()! [schemeName]!;
-    }
+    public static Scheme GetScheme (string schemeName) { return GetSchemesForCurrentTheme ()! [schemeName]!; }
 
     /// <summary>
     ///     Gets the name of the specified <see cref="Schemes"/>. Will throw an exception if <paramref name="schemeName"/>
     ///     is not a built-in Scheme.
     /// </summary>
     /// <param name="schemeName"></param>
     /// <returns>The name of scheme.</returns>
-    public static string? SchemesToSchemeName (Schemes schemeName)
-    {
-        return Enum.GetName (typeof (Schemes), schemeName);
-    }
+    public static string? SchemesToSchemeName (Schemes schemeName) { return Enum.GetName (typeof (Schemes), schemeName); }
 
     /// <summary>
     ///     Converts a string to a <see cref="Schemes"/> enum value.
@@ -158,11 +157,12 @@ public static Scheme GetScheme (string schemeName)
         {
             return value?.ToString ();
         }
+
         return null;
     }
 
     /// <summary>
-    ///     Get the dictionary schemes from the selected theme loaded from configuration.
+    ///     Get the dictionary of schemes from the current theme. Current means active.
     /// </summary>
     /// <returns></returns>
     public static Dictionary<string, Scheme?> GetSchemesForCurrentTheme ()
@@ -195,4 +195,14 @@ public static ImmutableList<string> GetSchemeNames ()
             return GetSchemes ()!.Keys.ToImmutableList ();
         }
     }
+
+    [RequiresUnreferencedCode ("Calls SetSchemes")]
+    [RequiresDynamicCode ("Calls SetSchemes")]
+    internal static void LoadToHardCodedDefaults ()
+    {
+        // BUGBUG: SchemeManager is broken and needs to be fixed to not have the hard coded schemes get overwritten.
+        // BUGBUG: This is a partial workaround
+        // BUGBUG: See https://github.com/gui-cs/Terminal.Gui/issues/4288
+        SetSchemes (GetHardCodedSchemes ()!.ToDictionary ());
+    }
 }
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ public override void Main ()`
`13`	`13`	`top.DrawComplete += (s, e) =>`
`14`	`14`	`{`
`15`	`15`	`// Forces reset _lineColsOffset because we're dealing with direct draw`
`16`		`- Application.ClearScreenNextIteration = true;`
	`16`	`+ Application.Top!.SetNeedsDraw ();`
`17`	`17`
`18`	`18`	`var i = -1;`
`19`	`19`	`top.AddStr ("Terminal.Gui only supports combining marks that normalize. See Issue #2616.");`
Original file line number	Diff line number	Diff line change
`@@ -12,13 +12,15 @@ public class Shortcuts : Scenario`
`12`	`12`	`public override void Main ()`
`13`	`13`	`{`
`14`	`14`	`Application.Init ();`
	`15`	`+ var quitKey = Application.QuitKey;`
`15`	`16`	`Window app = new ();`
`16`	`17`
`17`	`18`	`app.Loaded += App_Loaded;`
`18`	`19`
`19`	`20`	`Application.Run (app);`
`20`	`21`	`app.Dispose ();`
`21`	`22`	`Application.Shutdown ();`
	`23`	`+ Application.QuitKey = quitKey;`
`22`	`24`	`}`
`23`	`25`
`24`	`26`	`// Setting everything up in Loaded handler because we change the`
Original file line number	Diff line number	Diff line change
`@@ -172,7 +172,7 @@ public override void Main ()`
`172`	`172`	`else`
`173`	`173`	`{`
`174`	`174`	`appWindow.Remove (allViewsView);`
`175`		`- allViewsView.Dispose ();`
	`175`	`+ allViewsView!.Dispose ();`
`176`	`176`	`allViewsView = null;`
`177`	`177`
`178`	`178`	`appWindow.Add (viewFrame);`
Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ public static class ConfigurationManager`
`51`	`51`	`{`
`52`	`52`	`/// <summary>The backing property for <see cref="Settings"/> (config settings of <see cref="SettingsScope"/>).</summary>`
`53`	`53`	`/// <remarks>`
`54`		`- /// Is <see langword="null"/> until <see cref="ResetToCurrentValues"/> is called. Gets set to a new instance by`
	`54`	`+ /// Is <see langword="null"/> until <see cref="UpdateToCurrentValues"/> is called. Gets set to a new instance by`
`55`	`55`	`/// deserialization`
`56`	`56`	`/// (see <see cref="Load"/>).`
`57`	`57`	`/// </remarks>`
`@@ -117,15 +117,19 @@ internal static bool IsInitialized ()`
`117`	`117`	`}`
`118`	`118`	`}`
`119`	`119`
	`120`	`+ // TODO: Find a way to make this cache truly read-only at the leaf node level.`
	`121`	`+ // TODO: Right now, the dictionary is frozen, but the ConfigProperty instances can still be modified`
	`122`	`+ // TODO: if the PropertyValue is a reference type.`
	`123`	`+ // TODO: See https://github.com/gui-cs/Terminal.Gui/issues/4288`
`120`	`124`	`/// <summary>`
`121`	`125`	`/// A cache of all<see cref="ConfigurationPropertyAttribute"/> properties and their hard coded values.`
`122`	`126`	`/// </summary>`
`123`	`127`	`/// <remarks>Is <see langword="null"/> until <see cref="Initialize"/> is called.</remarks>`
`124`	`128`	`#pragma warning disable IDE1006 // Naming Styles`
`125`	`129`	`internal static FrozenDictionary<string, ConfigProperty>? _hardCodedConfigPropertyCache;`
	`130`	`+`
`126`	`131`	`private static readonly object _hardCodedConfigPropertyCacheLock = new ();`
`127`	`132`	`#pragma warning restore IDE1006 // Naming Styles`
`128`		`-`
`129`	`133`	`internal static FrozenDictionary<string, ConfigProperty>? GetHardCodedConfigPropertyCache ()`
`130`	`134`	`{`
`131`	`135`	`lock (_hardCodedConfigPropertyCacheLock)`
`@@ -183,7 +187,7 @@ internal static void Initialize ()`
`183`	`187`	`lock (_uninitializedConfigPropertiesCacheCacheLock)`
`184`	`188`	`{`
`185`	`189`	`// _allConfigProperties: for ordered, iterable access (LINQ-friendly)`
`186`		`- // _frozenConfigPropertyCache: for high-speed key lookup (frozen)`
	`190`	`+ // _hardCodedConfigPropertyCache: for high-speed key lookup (frozen)`
`187`	`191`
`188`	`192`	`// Note GetAllConfigProperties returns a new instance and all the properties !HasValue and Immutable.`
`189`	`193`	`_uninitializedConfigPropertiesCache = ConfigProperty.GetAllConfigProperties ();`
`@@ -209,6 +213,11 @@ internal static void Initialize ()`
`209`	`213`	`}`
`210`	`214`
`211`	`215`	`LoadHardCodedDefaults ();`
	`216`	`+`
	`217`	`+ // BUGBUG: ThemeScope is broken and needs to be fixed to not have the hard coded schemes get overwritten.`
	`218`	`+ // BUGBUG: This a partial workaround.`
	`219`	`+ // BUGBUG: See https://github.com/gui-cs/Terminal.Gui/issues/4288`
	`220`	`+ ThemeManager.Themes? [ThemeManager.Theme]?.Apply ();`
`212`	`221`	`}`
`213`	`222`
`214`	`223`	`#endregion Initialization`
`@@ -291,6 +300,7 @@ public static void Disable (bool resetToHardCodedDefaults = false)`
`291`	`300`
`292`	`301`	`if (resetToHardCodedDefaults)`
`293`	`302`	`{`
	`303`	`+ // Calls Apply`
`294`	`304`	`ResetToHardCodedDefaults ();`
`295`	`305`	`}`
`296`	`306`	`}`
`@@ -299,16 +309,17 @@ public static void Disable (bool resetToHardCodedDefaults = false)`
`299`	`309`
`300`	`310`	`#region Reset`
`301`	`311`
`302`		- // `Reset` - Reset the configuration to either the current values or the hard-coded defaults.
`303`		`- // Resetting does not load the configuration; it only resets the configuration to the default values.`
	`312`	+ // `Update` - Updates the configuration from either the current values or the hard-coded defaults.
	`313`	`+ // Updating does not load the configuration; it only updates the configuration to the values currently`
	`314`	`+ // in the static ConfigProperties.`
`304`	`315`
`305`	`316`	`/// <summary>`
`306`		`- /// INTERNAL: Resets <see cref="ConfigurationManager"/>. Loads settings from the current`
	`317`	`+ /// INTERNAL: Updates <see cref="ConfigurationManager"/> to the settings from the current`
`307`	`318`	`/// values of the static <see cref="ConfigurationPropertyAttribute"/> properties.`
`308`	`319`	`/// </summary>`
`309`	`320`	`[RequiresUnreferencedCode ("AOT")]`
`310`	`321`	`[RequiresDynamicCode ("AOT")]`
`311`		`- internal static void ResetToCurrentValues ()`
	`322`	`+ internal static void UpdateToCurrentValues ()`
`312`	`323`	`{`
`313`	`324`	`if (!IsInitialized ())`
`314`	`325`	`{`
`@@ -327,13 +338,13 @@ internal static void ResetToCurrentValues ()`
`327`	`338`	`_settingsLockSlim.ExitWriteLock ();`
`328`	`339`	`}`
`329`	`340`
`330`		`- Settings!.LoadCurrentValues ();`
	`341`	`+ Settings!.UpdateToCurrentValues ();`
`331`	`342`	`ThemeManager.UpdateToCurrentValues ();`
`332`		`- AppSettings!.LoadCurrentValues ();`
	`343`	`+ AppSettings!.UpdateToCurrentValues ();`
`333`	`344`	`}`
`334`	`345`
`335`	`346`	`/// <summary>`
`336`		`- /// INTERNAL: Resets <see cref="ConfigurationManager"/>. Loads the hard-coded values of the`
	`347`	`+ /// INTERNAL: Loads the hard-coded values of the`
`337`	`348`	`/// <see cref="ConfigurationPropertyAttribute"/> properties and applies them.`
`338`	`349`	`/// </summary>`
`339`	`350`	`[RequiresUnreferencedCode ("AOT")]`
`@@ -374,7 +385,7 @@ internal static void LoadHardCodedDefaults ()`
`374`	`385`
`375`	`386`	`Settings = new ();`
`376`	`387`	`Settings!.LoadHardCodedDefaults ();`
`377`		`- ThemeManager.ResetToHardCodedDefaults ();`
	`388`	`+ ThemeManager.LoadHardCodedDefaults ();`
`378`	`389`	`AppSettings!.LoadHardCodedDefaults ();`
`379`	`390`	`}`
`380`	`391`
`@@ -447,10 +458,6 @@ public static void Load (ConfigLocations locations)`
`447`	`458`	`{`
`448`	`459`	`SourcesManager?.Load (Settings, $"~/.tui/{AppName}.{_configFilename}", ConfigLocations.AppHome);`
`449`	`460`	`}`
`450`		`-`
`451`		`- Settings!.Validate ();`
`452`		`- ThemeManager.Validate ();`
`453`		`- AppSettings!.Validate ();`
`454`	`461`	`}`
`455`	`462`
`456`	`463`	`// TODO: Rename to Loaded?`
`@@ -566,7 +573,7 @@ private static void OnApplied ()`
`566`	`573`
`567`	`574`	`[SuppressMessage ("Style", "IDE1006:Naming Styles", Justification = "<Pending>")]`
`568`	`575`	`internal static readonly SourceGenerationContext SerializerContext = new (`
`569`		`- new JsonSerializerOptions`
	`576`	`+ new()`
`570`	`577`	`{`
`571`	`578`	`// Be relaxed`
`572`	`579`	`ReadCommentHandling = JsonCommentHandling.Skip,`
`@@ -638,7 +645,7 @@ public static AppSettingsScope? AppSettings`
`638`	`645`	`if (!appSettingsConfigProperty.HasValue)`
`639`	`646`	`{`
`640`	`647`	`var appSettings = new AppSettingsScope ();`
`641`		`- appSettings.LoadCurrentValues ();`
	`648`	`+ appSettings.UpdateToCurrentValues ();`
`642`	`649`
`643`	`650`	`return appSettings;`
`644`	`651`	`}`
`@@ -710,8 +717,9 @@ public static void PrintJsonErrors ()`
`710`	`717`	`{`
`711`	`718`	`if (_jsonErrors.Length > 0)`
`712`	`719`	`{`
`713`		`- Console.WriteLine (@"Terminal.Gui ConfigurationManager encountered these errors while reading configuration files" +`
`714`		`- @"(set ThrowOnJsonErrors to have these caught during execution):");`
	`720`	`+ Console.WriteLine (`
	`721`	`+ @"Terminal.Gui ConfigurationManager encountered these errors while reading configuration files"`
	`722`	`+ + @"(set ThrowOnJsonErrors to have these caught during execution):");`
`715`	`723`	`Console.WriteLine (_jsonErrors.ToString ());`
`716`	`724`	`}`
`717`	`725`	`}`
`@@ -783,8 +791,10 @@ public static string GetHardCodedConfig ()`
`783`	`791`
`784`	`792`	`Debug.Assert (filtered is { });`
`785`	`793`
`786`		`- IEnumerable<KeyValuePair<string, ConfigProperty>> configPropertiesByScope = filtered as KeyValuePair<string, ConfigProperty> [] ?? filtered.ToArray ();`
	`794`	`+ IEnumerable<KeyValuePair<string, ConfigProperty>> configPropertiesByScope =`
	`795`	`+ filtered as KeyValuePair<string, ConfigProperty> [] ?? filtered.ToArray ();`
`787`	`796`	`Debug.Assert (configPropertiesByScope.All (v => !v.Value.HasValue));`
	`797`	`+`
`788`	`798`	`return configPropertiesByScope;`
`789`	`799`	`}`
`790`	`800`	`}`