save settings on form close only fixes #53

unitycoder · Apr 6, 2018 · f2d6339 · f2d6339
1 parent a392d06
commit f2d6339
Show file tree

Hide file tree

Showing 2 changed files with 46 additions and 38 deletions.
diff --git a/UnityLauncher/Form1.Designer.cs b/UnityLauncher/Form1.Designer.cs
diff --git a/UnityLauncher/Form1.cs b/UnityLauncher/Form1.cs
@@ -36,6 +36,8 @@ private void Form1_Load(object sender, EventArgs e)
             Start();
         }
 
+
+
         void Start()
         {
             SetStatus("Initializing ...");
@@ -110,9 +112,6 @@ void Start()
             // preselect grid
             gridRecent.Select();
 
-            // subscribe to columnwidthchange event, so that can save column sizes
-            this.gridRecent.ColumnWidthChanged += new System.Windows.Forms.DataGridViewColumnEventHandler(this.gridRecent_ColumnWidthChanged);
-
             // get previous version build info string
             // this string is release tag for latest release when this app was compiled
             using (Stream stream = Assembly.GetExecutingAssembly().GetManifestResourceStream("UnityLauncher." + "PreviousVersion.txt"))
@@ -887,35 +886,6 @@ private void btnOpenLogFolder_Click(object sender, EventArgs e)
             }
         }
 
-        private void gridRecent_ColumnWidthChanged(object sender, DataGridViewColumnEventArgs e)
-        {
-            List<int> gridWidths;
-            if (Properties.Settings.Default.gridColumnWidths != null)
-            {
-                gridWidths = new List<int>(Properties.Settings.Default.gridColumnWidths);
-            }
-            else
-            {
-                gridWidths = new List<int>();
-            }
-
-            // restore data grid view widths
-            var colum = gridRecent.Columns[0];
-            for (int i = 0; i < gridRecent.Columns.Count; ++i)
-            {
-                if (Properties.Settings.Default.gridColumnWidths != null && Properties.Settings.Default.gridColumnWidths.Length > i)
-                {
-                    gridWidths[i] = gridRecent.Columns[i].Width;
-                }
-                else
-                {
-                    gridWidths.Add(gridRecent.Columns[i].Width);
-                }
-            }
-            Properties.Settings.Default.gridColumnWidths = gridWidths.ToArray();
-            Properties.Settings.Default.Save();
-        }
-
         private void btnOpenUpdateWebsite_Click(object sender, EventArgs e)
         {
             var selected = gridUnityUpdates?.CurrentCell?.RowIndex;
@@ -1025,8 +995,16 @@ private void btnBrowseForProject_Click(object sender, EventArgs e)
             BrowseForExistingProjectFolder();
         }
 
+        private void Form1_FormClosing(object sender, FormClosingEventArgs e)
+        {
+            SaveSettingsOnExit();
+        }
+
         #endregion UI events
 
+
+
+
         // displays version selector to upgrade project
         void UpgradeProject()
         {
@@ -1237,5 +1215,37 @@ void BrowseForExistingProjectFolder()
                 gridRecent.Rows[0].Selected = true;
             }
         }
-    }
-}
+
+        private void SaveSettingsOnExit()
+        {
+            // save list column widths
+            List<int> gridWidths;
+            if (Properties.Settings.Default.gridColumnWidths != null)
+            {
+                gridWidths = new List<int>(Properties.Settings.Default.gridColumnWidths);
+            }
+            else
+            {
+                gridWidths = new List<int>();
+            }
+
+            // restore data grid view widths
+            var colum = gridRecent.Columns[0];
+            for (int i = 0; i < gridRecent.Columns.Count; ++i)
+            {
+                if (Properties.Settings.Default.gridColumnWidths != null && Properties.Settings.Default.gridColumnWidths.Length > i)
+                {
+                    gridWidths[i] = gridRecent.Columns[i].Width;
+                }
+                else
+                {
+                    gridWidths.Add(gridRecent.Columns[i].Width);
+                }
+            }
+            Properties.Settings.Default.gridColumnWidths = gridWidths.ToArray();
+            Properties.Settings.Default.Save();
+        }
+
+
+    } // class Form 
+} // namespace