[Mac] Initial Variations Implementation

Author: Dominique Louis

Xamarin.PropertyEditing.Mac/Controls/BasePointEditorControl.cs

@@ -16,7 +16,7 @@ internal abstract class BasePointEditorControl<T> : PropertyEditorControl<Proper
 		public override NSView FirstKeyView => XEditor;
 		public override NSView LastKeyView => YEditor.DecrementButton;
 
-		protected override nint BaseHeight => 33;
+		protected override nint BaseHeight => 36;
 
 		protected BasePointEditorControl (IHostResourceProvider hostResources)
 			: base (hostResources)

Xamarin.PropertyEditing.Mac/Controls/BaseRectangleEditorControl.cs

@@ -22,7 +22,7 @@ internal abstract class BaseRectangleEditorControl<T> : PropertyEditorControl<Pr
 		public override NSView FirstKeyView => XEditor;
 		public override NSView LastKeyView => HeightEditor.DecrementButton;
 
-		protected override nint BaseHeight => 66;
+		protected override nint BaseHeight => 68;
 
 		protected BaseRectangleEditorControl (IHostResourceProvider hostResources)
 			: base (hostResources)

Xamarin.PropertyEditing.Mac/Controls/EditorContainer.cs

@@ -151,21 +151,24 @@ private void UpdateVariations ()
 
 							// Hook into the delete button's frame change, so we can fire off some drawing. If there's a better way, let me know.
 							this.deleteVariantButton.AddObserver (this, new NSString (FrameChangedObservableProperty), NSKeyValueObservingOptions.New, IntPtr.Zero);
-						}
+						}
+
+						// Cache these before the loop
+						var variationBgColour = HostResources.GetNamedColor (NamedResources.FrameBoxButtonBackgroundColor);
 
 						NSView previousControl = this.deleteVariantButton;
 						foreach (PropertyVariationOption item in this.viewModelAsPropertyViewModel.Variation) {
-							var selectedVariationTextField = new UnfocusableTextField {
-								BackgroundColor = HostResources.GetNamedColor (NamedResources.FrameBoxButtonBackgroundColor),
+							var selectedVariationTextField = new UnfocusableTextField {
+								BackgroundColor = variationBgColour,
 								Bordered = false,
-								Font = NSFont.SystemFontOfSize (NSFont.SystemFontSizeForControlSize (NSControlSize.Small) - 1f),
+								Font = VariationOptionFont,
 								TranslatesAutoresizingMaskIntoConstraints = false,
 								StringValue = string.Format (" {0}: {1} ", item.Category, item.Name),
 							};
 
 							AddSubview (selectedVariationTextField);
-							AddConstraints (new[] {
-								NSLayoutConstraint.Create (selectedVariationTextField, NSLayoutAttribute.Top, NSLayoutRelation.Equal, this, NSLayoutAttribute.Top, 1f, 5f),
+							AddConstraints (new[] {
+								NSLayoutConstraint.Create (selectedVariationTextField, NSLayoutAttribute.Top, NSLayoutRelation.Equal, this, NSLayoutAttribute.Top, 1f, VariationBorderOffset),
 								NSLayoutConstraint.Create (selectedVariationTextField, NSLayoutAttribute.Left, NSLayoutRelation.Equal, previousControl, NSLayoutAttribute.Right, 1f, 6f),
 								NSLayoutConstraint.Create (selectedVariationTextField, NSLayoutAttribute.Height, NSLayoutRelation.Equal, 1f, 16f),
 							});
@@ -315,6 +318,8 @@ public NSColor LabelTextColor {
 		private VariationButton addVariantButton;
 		private const float treeLineLeftEdge = 14f;
 		private const float treeLineLeafIndent = 4f;
+		internal const float VariationBorderOffset = 5f;
+		internal static NSFont VariationOptionFont = NSFont.SystemFontOfSize (NSFont.SystemFontSizeForControlSize (NSControlSize.Small) - 1f);
 		private nfloat ViewVariationChildVerticalDrawPoint => this.deleteVariantButton.Frame.Top + 5;
 		private nfloat ViewVariationParentVerticalDrawPoint => this.addVariantButton.Frame.Top - 2;
 		private PropertyViewModel viewModelAsPropertyViewModel;

Xamarin.PropertyEditing.Mac/Controls/PointEditorControl.cs

@@ -10,8 +10,6 @@ namespace Xamarin.PropertyEditing.Mac
 {
 	internal abstract class PointEditorControl<T> : BasePointEditorControl<T>
 	{
-		protected override float HeightScaleFactor => 1.8f;
-
 		public PointEditorControl (IHostResourceProvider hostResources)
 			: base (hostResources)
 		{

Xamarin.PropertyEditing.Mac/Controls/PropertyEditorControl.cs

@@ -37,13 +37,12 @@ public IHostResourceProvider HostResources
 		public virtual bool IsDynamicallySized => false;
 		public const float BottomOffset = -2f;
 		protected virtual nint BaseHeight => 24;
-		protected virtual float HeightScaleFactor => 2.0f;
 
 		PropertyViewModel viewModel;
 		public PropertyViewModel ViewModel {
 			get { return this.viewModel; }
 			set {
-				if (this.ViewModel == value)
+				if (this.viewModel == value)
 					return;
 
 				PropertyViewModel oldModel = this.viewModel;
@@ -110,7 +109,7 @@ public void UpdateKeyViews ()
 		public virtual nint GetHeight (EditorViewModel vm)
 		{
 			if (vm is PropertyViewModel realVm && realVm.IsVariant) {
-				return (nint)(BaseHeight * HeightScaleFactor);
+				return (nint)(BaseHeight + EditorContainer.VariationOptionFont.BoundingRectForFont.Height + (EditorContainer.VariationBorderOffset * 2)); // * 2 for upper and lower borders
 			}
 
 			return BaseHeight;

Xamarin.PropertyEditing.Mac/Controls/RectangleEditorControl.cs

@@ -11,8 +11,6 @@ namespace Xamarin.PropertyEditing.Mac
 	internal abstract class RectangleEditorControl<T>
 		: BaseRectangleEditorControl<T>
 	{
-		protected override float HeightScaleFactor => 1.4f;
-
 		protected RectangleEditorControl (IHostResourceProvider hostResources)
 			: base (hostResources)
 		{