From 368c04fdb49e570c626ba16b6a68297dabd69c28 Mon Sep 17 00:00:00 2001
From: mega5800 <43865655+mega5800@users.noreply.github.com>
Date: Tue, 9 Apr 2024 02:32:35 +0100
Subject: [PATCH] feat: added ability to save and load app window size and
 location (#183)

* added ability to save and load app window size and location

* chore: empty to commit to trigger ci

---------

Co-authored-by: Scott Willeke <scott@willeke.com>
---
 src/LessMsi.Gui/LessMsi.Gui.csproj |  1 +
 src/LessMsi.Gui/MainForm.cs        | 64 +++++++++++++++++++++++++++---
 src/LessMsi.Gui/app.config         |  9 ++++-
 3 files changed, 67 insertions(+), 7 deletions(-)

diff --git a/src/LessMsi.Gui/LessMsi.Gui.csproj b/src/LessMsi.Gui/LessMsi.Gui.csproj
index 7f97869..773da62 100644
--- a/src/LessMsi.Gui/LessMsi.Gui.csproj
+++ b/src/LessMsi.Gui/LessMsi.Gui.csproj
@@ -47,6 +47,7 @@
       <Private>True</Private>
     </Reference>
     <Reference Include="System" />
+    <Reference Include="System.configuration" />
     <Reference Include="System.Core" />
     <Reference Include="System.Data" />
     <Reference Include="System.Drawing" />
diff --git a/src/LessMsi.Gui/MainForm.cs b/src/LessMsi.Gui/MainForm.cs
index 4619e3b..8909970 100644
--- a/src/LessMsi.Gui/MainForm.cs
+++ b/src/LessMsi.Gui/MainForm.cs
@@ -25,6 +25,7 @@
 using System;
 using System.Collections.Generic;
 using System.ComponentModel;
+using System.Configuration;
 using System.Drawing;
 using System.Globalization;
 using System.IO;
@@ -50,8 +51,15 @@ internal class MainForm : Form, IMainFormView
 		private ToolStripMenuItem searchFileToolStripMenuItem;
 		readonly static string[] AllowedDragDropExtensions = new[] { ".msi", ".msp" };
 
+        #region Form size and location members
+        private const string c_WindowWidthAttribute = "WindowWidth";
+        private const string c_WindowHeightAttribute = "WindowHeight";
 
-		public MainForm(string defaultInputFile)
+        private const string c_XPositionAttribute = "X_Position";
+        private const string c_YPositionAttribute = "Y_Position";
+        #endregion
+
+        public MainForm(string defaultInputFile)
 		{
 			InitializeComponent();
 			msiTableGrid.AutoGenerateColumns = false;
@@ -815,6 +823,8 @@ private void InitializeComponent()
             this.Name = "MainForm";
             this.Text = "Less MSIérables";
             this.FormClosing += new System.Windows.Forms.FormClosingEventHandler(this.MainForm_FormClosing);
+            this.FormClosed += new System.Windows.Forms.FormClosedEventHandler(this.MainForm_FormClosed);
+            this.Load += new System.EventHandler(this.MainForm_Load);
             this.DragDrop += new System.Windows.Forms.DragEventHandler(this.MainForm_DragDrop);
             this.DragEnter += new System.Windows.Forms.DragEventHandler(this.MainForm_DragEnter);
             this.tabs.ResumeLayout(false);
@@ -1073,9 +1083,52 @@ private void cboStream_SelectedValueChanged(object sender, EventArgs e)
 			this.Presenter.OnSelectedStreamChanged();
 		}
 
-		#endregion
+        private void MainForm_Load(object sender, EventArgs e)
+        {
+            int windowWidth = getIntValueFromConfiguration(c_WindowWidthAttribute, MinimumSize.Width);
+            int windowHeight = getIntValueFromConfiguration(c_WindowHeightAttribute, MinimumSize.Height);
+
+            int xPosition = getIntValueFromConfiguration(c_XPositionAttribute, 0);
+            int yPosition = getIntValueFromConfiguration(c_YPositionAttribute, 0);
+
+            Size = new Size(windowWidth, windowHeight);
+            Location = new Point(xPosition, yPosition);
+        }
 
-		private ExtractionProgressDialog BeginShowingProgressDialog()
+        private void MainForm_FormClosed(object sender, FormClosedEventArgs e)
+        {
+            updateConfiguration(c_WindowWidthAttribute, Size.Width.ToString());
+            updateConfiguration(c_WindowHeightAttribute, Size.Height.ToString());
+
+            updateConfiguration(c_XPositionAttribute, Location.X.ToString());
+            updateConfiguration(c_YPositionAttribute, Location.Y.ToString());
+        }
+        #endregion
+
+        #region Form size and location methods
+        private int getIntValueFromConfiguration(string i_SettingName, int i_MinValue)
+        {
+            int intValue = 0;
+            string rawValue = ConfigurationManager.AppSettings.Get(i_SettingName);
+
+            int.TryParse(rawValue, out intValue);
+            intValue = Math.Max(intValue, i_MinValue);
+
+            return intValue;
+        }
+
+        private void updateConfiguration(string i_SettingName, string i_SettingValue)
+        {
+            Configuration config = ConfigurationManager.OpenExeConfiguration(Application.ExecutablePath);
+
+            config.AppSettings.Settings.Remove(i_SettingName);
+            config.AppSettings.Settings.Add(i_SettingName, i_SettingValue);
+
+            config.Save(ConfigurationSaveMode.Minimal);
+        }
+        #endregion
+
+        private ExtractionProgressDialog BeginShowingProgressDialog()
 		{
 			var progressDialog = new ExtractionProgressDialog(this);
 			progressDialog.Show();
@@ -1101,6 +1154,5 @@ protected bool IsFileTabSelected
 				return tabs.SelectedTab == tabExtractFiles;
 			}
 		}
-
-	}
-}
+    }
+}
\ No newline at end of file
diff --git a/src/LessMsi.Gui/app.config b/src/LessMsi.Gui/app.config
index 88fe64b..c843c11 100644
--- a/src/LessMsi.Gui/app.config
+++ b/src/LessMsi.Gui/app.config
@@ -4,4 +4,11 @@
 	<supportedRuntime version="v4.0"/>
 	<supportedRuntime version="v4.5"/>
 </startup>
-</configuration>
+  <appSettings>
+    <add key="WindowWidth" value="0"/>
+    <add key="WindowHeight" value="0"/>
+
+    <add key="X_Position" value="10"/>
+    <add key="Y_Position" value="10"/>
+  </appSettings>
+</configuration>
\ No newline at end of file