Merge pull request #590 from xamarin/dominique-Fix587

[Mac] Move appearance setting calls outside ViewDidChangeEffectiveApp…

1 files changed, 9 insertions, 3 deletions

diff --git a/Xamarin.PropertyEditing.Mac/Controls/Custom/TabButton.cs b/Xamarin.PropertyEditing.Mac/Controls/Custom/TabButton.cs
index e185105..ad18e31 100644
--- a/Xamarin.PropertyEditing.Mac/Controls/Custom/TabButton.cs
+++ b/Xamarin.PropertyEditing.Mac/Controls/Custom/TabButton.cs
@@ -24,7 +24,7 @@ namespace Xamarin.PropertyEditing.Mac
 			this.hostResource = hostResource;
 			this.imageName = imageName;
 
-			ViewDidChangeEffectiveAppearance ();
+			AppearanceChanged ();
 		}
 
 		public event EventHandler Clicked;
@@ -48,10 +48,11 @@ namespace Xamarin.PropertyEditing.Mac
 			}
 		}
 
-		public override void ViewDidChangeEffectiveAppearance ()
+		public sealed override void ViewDidChangeEffectiveAppearance ()
 		{
 			base.ViewDidChangeEffectiveAppearance ();
-			Image = this.hostResource.GetNamedImage (this.imageName);
+
+			AppearanceChanged ();
 		}
 
 		public override void DrawRect (CGRect dirtyRect)
@@ -66,6 +67,11 @@ namespace Xamarin.PropertyEditing.Mac
 			path.Fill ();
 		}
 
+		private void AppearanceChanged ()
+		{
+			Image = this.hostResource.GetNamedImage (this.imageName);
+		}
+
 		private readonly string imageName;
 		private readonly IHostResourceProvider hostResource;
 		private bool selected;