Merge #2729 Auto size yes no dialog

CHANGELOG.md

@@ -15,6 +15,7 @@ All notable changes to this project will be documented in this file.
 - [GUI] Fix size of relationships list (#2728 by: HebaruSan; reviewed: DasSkelett)
 - [GUI] Don't apply previous change set to checkboxes after successful install (#2730 by: HebaruSan; reviewed: DasSkelett)
 - [Core] Compare RelationshipDescriptors with Equals instead of Same (#2735 by: HebaruSan; reviewed: DasSkelett)
+- [GUI] Auto size yes no dialog (#2729 by: HebaruSan; reviewed: DasSkelett)
 
 ## v1.26.0
 

GUI/YesNoDialog.cs

@@ -1,8 +1,10 @@
+using System;
+using System.Drawing;
 using System.Windows.Forms;
 
 namespace CKAN
 {
-    public partial class YesNoDialog : FormCompatibility
+    public partial class YesNoDialog : Form
     {
         public YesNoDialog()
         {
@@ -11,13 +13,31 @@ public YesNoDialog()
 
         public DialogResult ShowYesNoDialog(string text)
         {
-            Util.Invoke(DescriptionLabel, () => DescriptionLabel.Text = text);
+            Util.Invoke(DescriptionLabel, () =>
+            {
+                DescriptionLabel.Text = text;
+                ClientSize = new Size(ClientSize.Width, StringHeight(text, ClientSize.Width - 25) + 2 * 54);
+            });
+
             return ShowDialog();
         }
 
+        /// <summary>
+        /// Simple syntactic sugar around Graphics.MeasureString
+        /// </summary>
+        /// <param name="text">String to measure size of</param>
+        /// <param name="maxWidth">Number of pixels allowed horizontally</param>
+        /// <returns>
+        /// Number of pixels needed vertically to fit the string
+        /// </returns>
+        private int StringHeight(string text, int maxWidth)
+        {
+            return (int)CreateGraphics().MeasureString(text, DescriptionLabel.Font, maxWidth).Height;
+        }
+
         public void HideYesNoDialog()
         {
             Util.Invoke(this, Close);
         }
     }
-}
+}