[DYN-6621] Save "Don't show again" setting on splash screen close (#14866) (#15018)

mjkkirschner · zeusongit · web-flow · commit cdf8c5a9585b · 2024-03-14T21:50:46.000-04:00
* Update SplashScreen.xaml.cs

* Update SplashScreen.xaml.cs

Co-authored-by: Ashish Aggarwal &lt;ashish.aggarwal@autodesk.com&gt;
diff --git a/src/DynamoCoreWpf/Views/SplashScreen/SplashScreen.xaml.cs b/src/DynamoCoreWpf/Views/SplashScreen/SplashScreen.xaml.cs
@@ -475,9 +475,15 @@ private static bool IsValidPreferencesFile(string filePath)
         /// <summary>
         /// If the user wants to close the window, we shutdown the application and don't launch Dynamo
         /// </summary>
-        internal void CloseWindow()
+        /// <param name="isCheckboxChecked">If true, the user has chosen to not show splash screen on next run.</param>
+        internal void CloseWindow(bool isCheckboxChecked = false)
         {
             CloseWasExplicit = true;
+            if (viewModel != null && isCheckboxChecked)
+            {
+                viewModel.PreferenceSettings.EnableStaticSplashScreen = !isCheckboxChecked;
+            }
+
             if (Application.Current != null)
             {
                 Application.Current.Shutdown();
@@ -519,6 +525,9 @@ enum ImportStatus
         success
     }
 
+    /// <summary>
+    /// This class is used to expose the methods that can be called from the webview2 component, SplashScreen.
+    /// </summary>
     [ClassInterface(ClassInterfaceType.AutoDual)]
     [ComVisible(true)]
     public class ScriptObject
@@ -528,7 +537,12 @@ public class ScriptObject
         readonly Func<bool> RequestSignIn;
         readonly Func<bool> RequestSignOut;
         readonly Action RequestCloseWindow;
+        readonly Action<bool> RequestCloseWindowPreserve;
 
+        /// <summary>
+        /// [Obsolete] Constructor for ScriptObject
+        /// </summary>
+        [Obsolete]
         public ScriptObject(Action<bool> requestLaunchDynamo, Action<string> requestImportSettings, Func< bool> requestSignIn, Func<bool> requestSignOut, Action requestCloseWindow)
         {
             RequestLaunchDynamo = requestLaunchDynamo;
@@ -537,6 +551,17 @@ public ScriptObject(Action<bool> requestLaunchDynamo, Action<string> requestImpo
             RequestSignOut = requestSignOut;
             RequestCloseWindow = requestCloseWindow;
         }
+        /// <summary>
+        /// Constructor for ScriptObject with an overload for close window method, to preserve "Don't show again" setting on splash screen on explicit close event.
+        /// </summary>
+        public ScriptObject(Action<bool> requestLaunchDynamo, Action<string> requestImportSettings, Func<bool> requestSignIn, Func<bool> requestSignOut, Action<bool> requestCloseWindow)
+        {
+            RequestLaunchDynamo = requestLaunchDynamo;
+            RequestImportSettings = requestImportSettings;
+            RequestSignIn = requestSignIn;
+            RequestSignOut = requestSignOut;
+            RequestCloseWindowPreserve = requestCloseWindow;
+        }
 
         public void LaunchDynamo(bool showScreenAgain)
         {
@@ -560,5 +585,9 @@ public void CloseWindow()
         {
             RequestCloseWindow();
         }
+        public void CloseWindowPreserve(bool isCheckboxChecked)
+        {
+            RequestCloseWindowPreserve(isCheckboxChecked);
+        }
     }
 }
