Add option to disable automatic resize of main window (squashed PR #4011)

* Add option to disable automatic main window resize

* Add option to window size submenu

* Rename parameter to `forceWindowResize`

* Add XML doc comments

* Save main window size

* Make `Config.MainWndx `/`MainWndy` nullable

* Reduce total diff size

* Move cast inside of `foreach`

* Remove obsolete workaround

Position isn't saved anymore when minimized

* Combine config values into `Point` and `Size`

* Add `Point` and `Size` to known good serializable types

* Remove doc comment from implementation

Author: kalimag

src/BizHawk.Client.Common/Api/Classes/EmuClientApi.cs

@@ -197,7 +197,7 @@ public void SetWindowSize(int size)
 			if (size == 1 || size == 2 || size == 3 || size == 4 || size == 5 || size == 10)
 			{
 				_config.SetWindowScaleFor(Emulator.SystemId, size);
-				_mainForm.FrameBufferResized();
+				_mainForm.FrameBufferResized(forceWindowResize: true);
 				_displayManager.OSD.AddMessage($"Window size set to {size}x");
 			}
 			else

src/BizHawk.Client.Common/IMainFormForApi.cs

@@ -53,7 +53,8 @@ public interface IMainFormForApi
 		/// <remarks>only referenced from <c>EmuClientApi</c></remarks>
 		void FrameAdvance(bool discardApiHawkSurfaces = true);
 
-		void FrameBufferResized();
+		/// <param name="forceWindowResize">Override <see cref="Common.Config.ResizeWithFramebuffer"/></param>
+		void FrameBufferResized(bool forceWindowResize = false);
 
 		void FrameSkipMessage();
 

src/BizHawk.Client.Common/config/Config.cs

@@ -1,4 +1,5 @@
 using System.Collections.Generic;
+using System.Drawing;
 using System.IO;
 
 using BizHawk.Bizware.Graphics;
@@ -139,8 +140,8 @@ public void SetWindowScaleFor(string sysID, int windowScale)
 		public bool MainFormStayOnTop { get; set; }
 		public bool StartPaused { get; set; }
 		public bool StartFullscreen { get; set; }
-		public int MainWndx { get; set; } = -1; // Negative numbers will be ignored
-		public int MainWndy { get; set; } = -1;
+		public Point? MainWindowPosition { get; set; }
+		public Size? MainWindowSize { get; set; }
 		public bool RunInBackground { get; set; } = true;
 		public bool AcceptBackgroundInput { get; set; }
 		public bool AcceptBackgroundInputControllerOnly { get; set; }
@@ -277,6 +278,11 @@ public void SetWindowScaleFor(string sysID, int windowScale)
 		public int DispCropRight { get; set; } = 0;
 		public int DispCropBottom { get; set; } = 0;
 
+		/// <summary>
+		/// Automatically resize main window when framebuffer size changes (default behavior)
+		/// </summary>
+		public bool ResizeWithFramebuffer { get; set; } = true;
+
 		// Sound options
 		public ESoundOutputMethod SoundOutputMethod { get; set; } = HostCapabilityDetector.HasXAudio2 ? ESoundOutputMethod.XAudio2 : ESoundOutputMethod.OpenAL;
 

src/BizHawk.Client.EmuHawk/IMainFormForTools.cs

@@ -58,7 +58,8 @@ public interface IMainFormForTools : IDialogController
 		void FrameAdvance(bool discardApiHawkSurfaces = true);
 
 		/// <remarks>only referenced from <see cref="LuaConsole"/></remarks>
-		void FrameBufferResized();
+		/// <param name="forceWindowResize">Override <see cref="Common.Config.ResizeWithFramebuffer"/></param>
+		void FrameBufferResized(bool forceWindowResize = false);
 
 		/// <remarks>only referenced from <see cref="BasicBot"/></remarks>
 		bool LoadQuickSave(int slot, bool suppressOSD = false);

src/BizHawk.Client.EmuHawk/MainForm.Designer.cs

@@ -611,16 +611,27 @@ private void ViewSubMenu_DropDownOpened(object sender, EventArgs e)
 		private void WindowSizeSubMenu_DropDownOpened(object sender, EventArgs e)
 		{
 			var windowScale = Config.GetWindowScaleFor(Emulator.SystemId);
-			foreach (ToolStripMenuItem item in WindowSizeSubMenu.DropDownItems)
+			foreach (var item in WindowSizeSubMenu.DropDownItems)
 			{
-				item.Checked = (int) item.Tag == windowScale;
+				// filter out separators
+				if (item is ToolStripMenuItem menuItem && menuItem.Tag is int itemScale)
+				{
+					menuItem.Checked = itemScale == windowScale && Config.ResizeWithFramebuffer;
+				}
 			}
+			DisableResizeWithFramebufferMenuItem.Checked = !Config.ResizeWithFramebuffer;
+		}
+
+		private void DisableResizeWithFramebufferMenuItem_Click(object sender, EventArgs e)
+		{
+			Config.ResizeWithFramebuffer = !DisableResizeWithFramebufferMenuItem.Checked;
+			FrameBufferResized();
 		}
 
 		private void WindowSize_Click(object sender, EventArgs e)
 		{
 			Config.SetWindowScaleFor(Emulator.SystemId, (int) ((ToolStripMenuItem) sender).Tag);
-			FrameBufferResized();
+			FrameBufferResized(forceWindowResize: true);
 		}
 
 		private void SwitchToFullscreenMenuItem_Click(object sender, EventArgs e)

src/BizHawk.Client.EmuHawk/MainForm.Events.cs

@@ -61,9 +61,7 @@ private void MainForm_Load(object sender, EventArgs e)
 		{	
 			UpdateWindowTitle();
 
-			ToolStripItem[] CreateWindowSizeFactorSubmenus()
 			{
-				var items = new ToolStripItem[WINDOW_SCALE_MAX];
 				for (int i = 1; i <= WINDOW_SCALE_MAX; i++)
 				{
 					long quotient = Math.DivRem(i, 10, out long remainder);
@@ -73,11 +71,9 @@ ToolStripItem[] CreateWindowSizeFactorSubmenus()
 						Text = $"{(quotient > 0 ? quotient : "")}&{remainder}x"
 					};
 					temp.Click += this.WindowSize_Click;
-					items[i - 1] = temp;
+					WindowSizeSubMenu.DropDownItems.Insert(i - 1, temp);
 				}
-				return items;
 			}
-			WindowSizeSubMenu.DropDownItems.AddRange(CreateWindowSizeFactorSubmenus());
 
 			foreach (var (appliesTo, coreNames) in Config.CorePickerUIData)
 			{
@@ -603,20 +599,17 @@ _argParser.SocketAddress is var (socketIP, socketPort)
 			CheatList.Changed += Tools.UpdateCheatRelatedTools;
 			RewireSound();
 
-			// Workaround for windows, location is -32000 when minimized, if they close it during this time, that's what gets saved
-			if (Config.MainWndx == -32000)
-			{
-				Config.MainWndx = 0;
-			}
-
-			if (Config.MainWndy == -32000)
+			if (Config.SaveWindowPosition)
 			{
-				Config.MainWndy = 0;
-			}
+				if (Config.MainWindowPosition is Point position)
+				{
+					Location = position;
+				}
 
-			if (Config.MainWndx != -1 && Config.MainWndy != -1 && Config.SaveWindowPosition)
-			{
-				Location = new Point(Config.MainWndx, Config.MainWndy);
+				if (Config.MainWindowSize is Size size && !Config.ResizeWithFramebuffer)
+				{
+					Size = size;
+				}
 			}
 
 			if (Config.MainFormStayOnTop) TopMost = true;
@@ -1395,8 +1388,12 @@ public void TakeScreenshot(string path)
 			AddOnScreenMessage($"{fi.Name} saved.");
 		}
 
-		public void FrameBufferResized()
+		public void FrameBufferResized(bool forceWindowResize = false)
 		{
+			if (!Config.ResizeWithFramebuffer && !forceWindowResize)
+			{
+				return;
+			}
 			// run this entire thing exactly twice, since the first resize may adjust the menu stacking
 			for (int i = 0; i < 2; i++)
 			{
@@ -2422,20 +2419,16 @@ private void SaveConfig(string path = "")
 		{
 			if (Config.SaveWindowPosition)
 			{
-				if (Config.MainWndx != -32000) // When minimized location is -32000, don't save this into the config file!
-				{
-					Config.MainWndx = Location.X;
-				}
-
-				if (Config.MainWndy != -32000)
+				if (WindowState is FormWindowState.Normal)
 				{
-					Config.MainWndy = Location.Y;
+					Config.MainWindowPosition = Location;
+					Config.MainWindowSize = Size;
 				}
 			}
 			else
 			{
-				Config.MainWndx = -1;
-				Config.MainWndy = -1;
+				Config.MainWindowPosition = null;
+				Config.MainWindowSize = null;
 			}
 
 			Config.LastWrittenFrom = VersionInfo.MainVersion;
@@ -2591,7 +2584,7 @@ private void IncreaseWindowSize()
 				Config.SetWindowScaleFor(Emulator.SystemId, windowScale);
 			}
 			AddOnScreenMessage($"Screensize set to {windowScale}x");
-			FrameBufferResized();
+			FrameBufferResized(forceWindowResize: true);
 		}
 
 		private void DecreaseWindowSize()
@@ -2603,7 +2596,7 @@ private void DecreaseWindowSize()
 				Config.SetWindowScaleFor(Emulator.SystemId, windowScale);
 			}
 			AddOnScreenMessage($"Screensize set to {windowScale}x");
-			FrameBufferResized();
+			FrameBufferResized(forceWindowResize: true);
 		}
 
 		private static readonly int[] SpeedPercents = { 1, 3, 6, 12, 25, 50, 75, 100, 150, 200, 300, 400, 800, 1600, 3200, 6400 };

src/BizHawk.Client.EmuHawk/MainForm.cs

@@ -1,4 +1,5 @@
 using System.Collections.Generic;
+using System.Drawing;
 using System.Reflection;
 
 using BizHawk.Client.Common;
@@ -32,8 +33,10 @@ public sealed class SerializationStabilityTests
 			typeof(List<>),
 			typeof(Nullable<>),
 			typeof(object),
+			typeof(Point),
 			typeof(Queue<>),
 			typeof(float),
+			typeof(Size),
 			typeof(string),
 		};