From f828641388446c1f75093f639aa262908cfcdc7f Mon Sep 17 00:00:00 2001
From: feos <feos-theos@yandex.ru>
Date: Sat, 10 Feb 2018 16:53:25 +0300
Subject: [PATCH] core config: dynamically resize description field to fit text
 fix #1091

---
 .../config/GenericCoreConfig.cs               | 34 +++++++++++++++++++
 .../Computers/Commodore64/C64.ISettable.cs    | 18 +++++++---
 2 files changed, 48 insertions(+), 4 deletions(-)

diff --git a/BizHawk.Client.EmuHawk/config/GenericCoreConfig.cs b/BizHawk.Client.EmuHawk/config/GenericCoreConfig.cs
index d8c19d9644..f5891e284a 100644
--- a/BizHawk.Client.EmuHawk/config/GenericCoreConfig.cs
+++ b/BizHawk.Client.EmuHawk/config/GenericCoreConfig.cs
@@ -1,5 +1,7 @@
 using System;
 using System.Windows.Forms;
+using System.Reflection;
+using System.ComponentModel;
 using BizHawk.Client.Common;
 using BizHawk.Emulation.Common;
 
@@ -31,6 +33,7 @@ namespace BizHawk.Client.EmuHawk
 			if (_s != null)
 			{
 				propertyGrid1.SelectedObject = _s;
+				ChangeDescriptionHeight(propertyGrid1);
 			}
 			else
 			{
@@ -40,6 +43,7 @@ namespace BizHawk.Client.EmuHawk
 			if (_ss != null)
 			{
 				propertyGrid2.SelectedObject = _ss;
+				ChangeDescriptionHeight(propertyGrid2);
 			}
 			else
 			{
@@ -57,6 +61,36 @@ namespace BizHawk.Client.EmuHawk
 		{
 		}
 
+		private static void ChangeDescriptionHeight(PropertyGrid grid)
+		{
+			if (grid == null)
+				throw new ArgumentNullException("grid");
+
+			int maxlen = 0;
+			string desc = "";
+
+			foreach (PropertyDescriptor property in TypeDescriptor.GetProperties(grid.SelectedObject))
+			{
+				if (property.Description.Length > maxlen)
+				{
+					maxlen = property.Description.Length;
+					desc = property.Description;
+				}
+			}
+
+			foreach (Control control in grid.Controls)
+			{
+				if (control.GetType().Name == "DocComment")
+				{
+					FieldInfo field = control.GetType().GetField("userSized", BindingFlags.Instance | BindingFlags.NonPublic);
+					field.SetValue(control, true);
+					int height = (int)System.Drawing.Graphics.FromHwnd(control.Handle).MeasureString(desc, control.Font, grid.Width).Height;
+					control.Height = Math.Max(20, height) + 16; // magic for now
+					return;
+				}
+			}
+		}
+
 		private void OkBtn_Click(object sender, EventArgs e)
 		{
 			if (_s != null && settingschanged)
diff --git a/BizHawk.Emulation.Cores/Computers/Commodore64/C64.ISettable.cs b/BizHawk.Emulation.Cores/Computers/Commodore64/C64.ISettable.cs
index a1c4c358a1..a87e50610a 100644
--- a/BizHawk.Emulation.Cores/Computers/Commodore64/C64.ISettable.cs
+++ b/BizHawk.Emulation.Cores/Computers/Commodore64/C64.ISettable.cs
@@ -52,22 +52,32 @@ namespace BizHawk.Emulation.Cores.Computers.Commodore64
 		public class C64SyncSettings
 		{
 			[DisplayName("VIC type")]
-			[Description("Set the type of video chip to use")]
+			[Description("Set the type of video chip to use\n" +
+				"PAL: ~50hz. All PAL games will expect this configuration.\n" +
+				"NTSC: ~60hz.This is the most common NTSC configuration. Every NTSC game should work with this configuration.\n" +
+				"NTSCOld: ~60hz.This was used in the very earliest systems and will exhibit problems on most modern games.\n" +
+				"Drean: ~60hz.This was manufactured for a very specific market and is not very compatible with timing sensitive games.\n")]
 			[DefaultValue(VicType.Pal)]
 			public VicType VicType { get; set; }
 
 			[DisplayName("SID type")]
-			[Description("Set the type of sound chip to use")]
+			[Description("Set the type of sound chip to use\n" +
+				"OldR2, OldR3, OldR4AR: Original 6581 SID chip.\n" +
+				"NewR5: Updated 8580 SID chip.\n" +
+				"")]
 			[DefaultValue(SidType.OldR2)]
 			public SidType SidType { get; set; }
 
 			[DisplayName("Tape drive type")]
-			[Description("Set the type of tape drive attached")]
+			[Description("Set the type of tape drive attached\n" +
+				"1531: Original Datasette device.")]
 			[DefaultValue(TapeDriveType.None)]
 			public TapeDriveType TapeDriveType { get; set; }
 
 			[DisplayName("Disk drive type")]
-			[Description("Set the type of disk drive attached")]
+			[Description("Set the type of disk drive attached\n" +
+				"1541: Original disk drive and ROM.\n" +
+				"1541 - II: Improved model with some ROM bugfixes.")]
 			[DefaultValue(DiskDriveType.None)]
 			public DiskDriveType DiskDriveType { get; set; }