Don't setup native defaults by default

dotnet · Jun 28, 2021 · 95630d6 · 95630d6
1 parent 8d3783b
commit 95630d6
Show file tree

Hide file tree

Showing 23 changed files with 33 additions and 56 deletions.
diff --git a/src/Core/src/Handlers/ActivityIndicator/ActivityIndicatorHandler.Windows.cs b/src/Core/src/Handlers/ActivityIndicator/ActivityIndicatorHandler.Windows.cs
@@ -11,11 +11,9 @@ public partial class ActivityIndicatorHandler : ViewHandler<IActivityIndicator,
 			Style = UI.Xaml.Application.Current.Resources["MauiActivityIndicatorStyle"] as UI.Xaml.Style
 		};
 
-		protected override void SetupDefaults(MauiActivityIndicator nativeView)
+		void SetupDefaults(MauiActivityIndicator nativeView)
 		{
 			_foregroundDefault = nativeView.GetForegroundCache();
-
-			base.SetupDefaults(nativeView);
 		}
 
 		public static void MapIsRunning(ActivityIndicatorHandler handler, IActivityIndicator activityIndicator)

diff --git a/src/Core/src/Handlers/Button/ButtonHandler.Android.cs b/src/Core/src/Handlers/Button/ButtonHandler.Android.cs
@@ -25,7 +25,7 @@ protected override AppCompatButton CreateNativeView()
 			return nativeButton;
 		}
 
-		protected override void SetupDefaults(AppCompatButton nativeView)
+		void SetupDefaults(AppCompatButton nativeView)
 		{
 			DefaultPadding = new Thickness(
 				nativeView.PaddingLeft,
@@ -34,8 +34,6 @@ protected override void SetupDefaults(AppCompatButton nativeView)
 				nativeView.PaddingBottom);
 
 			DefaultBackground = nativeView.Background;
-
-			base.SetupDefaults(nativeView);
 		}
 
 		protected override void ConnectHandler(AppCompatButton nativeView)

diff --git a/src/Core/src/Handlers/Button/ButtonHandler.Windows.cs b/src/Core/src/Handlers/Button/ButtonHandler.Windows.cs
@@ -14,13 +14,11 @@ public partial class ButtonHandler : ViewHandler<IButton, MauiButton>
 		protected override MauiButton CreateNativeView() 
 			=> new MauiButton();
 
-		protected override void SetupDefaults(MauiButton nativeView)
+		void SetupDefaults(MauiButton nativeView)
 		{
 			DefaultPadding = (UI.Xaml.Thickness)MauiWinUIApplication.Current.Resources["ButtonPadding"];
 			DefaultForeground = (UI.Xaml.Media.Brush)MauiWinUIApplication.Current.Resources["ButtonForegroundThemeBrush"];
 			DefaultBackground = (UI.Xaml.Media.Brush)MauiWinUIApplication.Current.Resources["ButtonBackgroundThemeBrush"];
-
-			base.SetupDefaults(nativeView);
 		}
 
 		protected override void ConnectHandler(MauiButton nativeView)

diff --git a/src/Core/src/Handlers/Button/ButtonHandler.iOS.cs b/src/Core/src/Handlers/Button/ButtonHandler.iOS.cs
@@ -37,13 +37,11 @@ protected override void DisconnectHandler(UIButton nativeView)
 			base.DisconnectHandler(nativeView);
 		}
 
-		protected override void SetupDefaults(UIButton nativeView)
+		void SetupDefaults(UIButton nativeView)
 		{
 			ButtonTextColorDefaultNormal = nativeView.TitleColor(UIControlState.Normal);
 			ButtonTextColorDefaultHighlighted = nativeView.TitleColor(UIControlState.Highlighted);
 			ButtonTextColorDefaultDisabled = nativeView.TitleColor(UIControlState.Disabled);
-
-			base.SetupDefaults(nativeView);
 		}
 
 		public static void MapText(ButtonHandler handler, IButton button)

diff --git a/src/Core/src/Handlers/DatePicker/DatePickerHandler.Android.cs b/src/Core/src/Handlers/DatePicker/DatePickerHandler.Android.cs
@@ -27,11 +27,11 @@ protected override MauiDatePicker CreateNativeView()
 			return mauiDatePicker;
 		}
 
-		protected override void SetupDefaults(MauiDatePicker nativeView)
+		void SetupDefaults(MauiDatePicker nativeView)
 		{
 			DefaultBackground = nativeView.Background;
 
-			base.SetupDefaults(nativeView);
+
 		}
 
 		internal DatePickerDialog? DatePickerDialog { get { return _dialog; } }

diff --git a/src/Core/src/Handlers/DatePicker/DatePickerHandler.Windows.cs b/src/Core/src/Handlers/DatePicker/DatePickerHandler.Windows.cs
@@ -20,11 +20,11 @@ protected override void DisconnectHandler(CalendarDatePicker nativeView)
 			nativeView.DateChanged -= DateChanged;
 		}
 
-		protected override void SetupDefaults(CalendarDatePicker nativeView)
+		void SetupDefaults(CalendarDatePicker nativeView)
 		{
 			_defaultForeground = nativeView.Foreground;
 
-			base.SetupDefaults(nativeView);
+
 		}
 
 		public static void MapFormat(DatePickerHandler handler, IDatePicker datePicker)

diff --git a/src/Core/src/Handlers/Editor/EditorHandler.Android.cs b/src/Core/src/Handlers/Editor/EditorHandler.Android.cs
@@ -44,9 +44,9 @@ protected override void DisconnectHandler(AppCompatEditText nativeView)
 			FocusChangeListener.Handler = null;
 		}
 
-		protected override void SetupDefaults(AppCompatEditText nativeView)
+		void SetupDefaults(AppCompatEditText nativeView)
 		{
-			base.SetupDefaults(nativeView);
+
 
 			DefaultTextColors = nativeView.TextColors;
 			DefaultPlaceholderTextColors = nativeView.HintTextColors;

diff --git a/src/Core/src/Handlers/Editor/EditorHandler.Windows.cs b/src/Core/src/Handlers/Editor/EditorHandler.Windows.cs
@@ -28,12 +28,12 @@ protected override void DisconnectHandler(MauiTextBox nativeView)
 			nativeView.LostFocus -= OnLostFocus;
 		}
 
-		protected override void SetupDefaults(MauiTextBox nativeView)
+		void SetupDefaults(MauiTextBox nativeView)
 		{
 			_placeholderDefaultBrush = nativeView.PlaceholderForeground;
 			_defaultPlaceholderColorFocusBrush = nativeView.PlaceholderForegroundFocusBrush;
 
-			base.SetupDefaults(nativeView);
+
 		}
 
 		public static void MapText(EditorHandler handler, IEditor editor)

diff --git a/src/Core/src/Handlers/Entry/EntryHandler.Android.cs b/src/Core/src/Handlers/Entry/EntryHandler.Android.cs
@@ -58,9 +58,9 @@ protected override void DisconnectHandler(AppCompatEditText nativeView)
 			ActionListener.Handler = null;
 		}
 
-		protected override void SetupDefaults(AppCompatEditText nativeView)
+		void SetupDefaults(AppCompatEditText nativeView)
 		{
-			base.SetupDefaults(nativeView);
+
 
 			ClearButtonDrawable = GetClearButtonDrawable();
 			DefaultTextColors = nativeView.TextColors;

diff --git a/src/Core/src/Handlers/Entry/EntryHandler.iOS.cs b/src/Core/src/Handlers/Entry/EntryHandler.iOS.cs
@@ -40,7 +40,7 @@ protected override void DisconnectHandler(MauiTextField nativeView)
 			nativeView.ShouldChangeCharacters -= OnShouldChangeCharacters;
 		}
 
-		protected override void SetupDefaults(MauiTextField nativeView)
+		void SetupDefaults(MauiTextField nativeView)
 		{
 			DefaultTextColor = nativeView.TextColor;
 		}

diff --git a/src/Core/src/Handlers/Label/LabelHandler.Android.cs b/src/Core/src/Handlers/Label/LabelHandler.Android.cs
@@ -18,7 +18,7 @@ public partial class LabelHandler : ViewHandler<ILabel, TextView>
 
 		protected override TextView CreateNativeView() => new TextView(Context);
 
-		protected override void SetupDefaults(TextView nativeView)
+		void SetupDefaults(TextView nativeView)
 		{
 			if (nativeView.TextColors == null)
 			{

diff --git a/src/Core/src/Handlers/Picker/PickerHandler.Android.cs b/src/Core/src/Handlers/Picker/PickerHandler.Android.cs
@@ -38,9 +38,9 @@ protected override void DisconnectHandler(MauiPicker nativeView)
 			base.DisconnectHandler(nativeView);
 		}
 
-		protected override void SetupDefaults(MauiPicker nativeView)
+		void SetupDefaults(MauiPicker nativeView)
 		{
-			base.SetupDefaults(nativeView);
+
 
 			DefaultBackground = nativeView.Background;
 			DefaultTitleColors = nativeView.HintTextColors;

diff --git a/src/Core/src/Handlers/Picker/PickerHandler.Windows.cs b/src/Core/src/Handlers/Picker/PickerHandler.Windows.cs
@@ -28,11 +28,11 @@ protected override void DisconnectHandler(MauiComboBox nativeView)
 			nativeView.SelectionChanged -= OnControlSelectionChanged;
 		}
 
-		protected override void SetupDefaults(MauiComboBox nativeView)
+		void SetupDefaults(MauiComboBox nativeView)
 		{
 			_defaultForeground = nativeView.Foreground;
 
-			base.SetupDefaults(nativeView);
+
 		}
 		void Reload()
 		{

diff --git a/src/Core/src/Handlers/SearchBar/SearchBarHandler.Android.cs b/src/Core/src/Handlers/SearchBar/SearchBarHandler.Android.cs
@@ -22,11 +22,11 @@ protected override SearchView CreateNativeView()
 			return searchView;
 		}
 
-		protected override void SetupDefaults(SearchView nativeView)
+		void SetupDefaults(SearchView nativeView)
 		{
 			DefaultBackground = nativeView.Background;
 
-			base.SetupDefaults(nativeView);
+
 		}
 
 		// This is a Android-specific mapping

diff --git a/src/Core/src/Handlers/SearchBar/SearchBarHandler.iOS.cs b/src/Core/src/Handlers/SearchBar/SearchBarHandler.iOS.cs
@@ -47,7 +47,7 @@ protected override void DisconnectHandler(UISearchBar nativeView)
 			base.DisconnectHandler(nativeView);
 		}
 
-		protected override void SetupDefaults(UISearchBar nativeView)
+		void SetupDefaults(UISearchBar nativeView)
 		{
 			_defaultTextColor = QueryEditor?.TextColor;
 
@@ -60,7 +60,7 @@ protected override void SetupDefaults(UISearchBar nativeView)
 				_cancelButtonTextColorDefaultDisabled = cancelButton.TitleColor(UIControlState.Disabled);
 			}
 
-			base.SetupDefaults(nativeView);
+
 		}
 
 		public static void MapText(SearchBarHandler handler, ISearchBar searchBar)

diff --git a/src/Core/src/Handlers/Slider/SliderHandler.Android.cs b/src/Core/src/Handlers/Slider/SliderHandler.Android.cs
@@ -37,7 +37,7 @@ protected override void DisconnectHandler(SeekBar nativeView)
 			nativeView.SetOnSeekBarChangeListener(null);
 		}
 
-		protected override void SetupDefaults(SeekBar nativeView)
+		void SetupDefaults(SeekBar nativeView)
 		{
 			DefaultThumbColorFilter = nativeView.Thumb?.GetColorFilter();
 			DefaultProgressTintMode = nativeView.ProgressTintMode;

diff --git a/src/Core/src/Handlers/Slider/SliderHandler.Windows.cs b/src/Core/src/Handlers/Slider/SliderHandler.Windows.cs
@@ -48,7 +48,7 @@ protected override void DisconnectHandler(MauiSlider nativeView)
 			_pointerReleasedHandler = null;
 		}
 
-		protected override void SetupDefaults(MauiSlider nativeView)
+		void SetupDefaults(MauiSlider nativeView)
 		{
 			DefaultForegroundColor = nativeView.Foreground;
 			DefaultBackgroundColor = nativeView.Background;

diff --git a/src/Core/src/Handlers/Slider/SliderHandler.iOS.cs b/src/Core/src/Handlers/Slider/SliderHandler.iOS.cs
@@ -29,7 +29,7 @@ protected override void DisconnectHandler(UISlider nativeView)
 			nativeView.RemoveTarget(OnTouchUpControlEvent, UIControlEvent.TouchUpInside | UIControlEvent.TouchUpOutside);
 		}
 
-		protected override void SetupDefaults(UISlider nativeView)
+		void SetupDefaults(UISlider nativeView)
 		{
 			DefaultMinTrackColor = nativeView.MinimumTrackTintColor;
 			DefaultMaxTrackColor = nativeView.MaximumTrackTintColor;

diff --git a/src/Core/src/Handlers/Switch/SwitchHandler.Android.cs b/src/Core/src/Handlers/Switch/SwitchHandler.Android.cs
@@ -30,7 +30,7 @@ protected override void DisconnectHandler(ASwitch nativeView)
 			nativeView.SetOnCheckedChangeListener(null);
 		}
 
-		protected override void SetupDefaults(ASwitch nativeView)
+		void SetupDefaults(ASwitch nativeView)
 		{
 			DefaultTrackColorStateList = nativeView.GetDefaultSwitchTrackColorStateList();
 			DefaultThumbColorStateList = nativeView.GetDefaultSwitchThumbColorStateList();

diff --git a/src/Core/src/Handlers/Switch/SwitchHandler.iOS.cs b/src/Core/src/Handlers/Switch/SwitchHandler.iOS.cs
@@ -29,7 +29,7 @@ protected override void DisconnectHandler(UISwitch nativeView)
 			nativeView.ValueChanged -= OnControlValueChanged;
 		}
 
-		protected override void SetupDefaults(UISwitch nativeView)
+		void SetupDefaults(UISwitch nativeView)
 		{
 			DefaultOnTrackColor = UISwitch.Appearance.OnTintColor;
 			DefaultOffTrackColor = nativeView.GetOffTrackColor();

diff --git a/src/Core/src/Handlers/TimePicker/TimePickerHandler.Android.cs b/src/Core/src/Handlers/TimePicker/TimePickerHandler.Android.cs
@@ -24,11 +24,11 @@ protected override MauiTimePicker CreateNativeView()
 			return _timePicker;
 		}
 
-		protected override void SetupDefaults(MauiTimePicker nativeView)
+		void SetupDefaults(MauiTimePicker nativeView)
 		{
 			DefaultBackground = nativeView.Background;
 
-			base.SetupDefaults(nativeView);
+
 		}
 
 		protected override void DisconnectHandler(MauiTimePicker nativeView)

diff --git a/src/Core/src/Handlers/TimePicker/TimePickerHandler.Windows.cs b/src/Core/src/Handlers/TimePicker/TimePickerHandler.Windows.cs
@@ -20,11 +20,11 @@ protected override void DisconnectHandler(TimePicker nativeView)
 			nativeView.TimeChanged -= OnControlTimeChanged;
 		}
 
-    protected override void SetupDefaults(TimePicker nativeView)
+    void SetupDefaults(TimePicker nativeView)
 		{
 			_defaultForeground = nativeView.Foreground;
 
-			base.SetupDefaults(nativeView);
+
 		}
 
 	  public static void MapFormat(TimePickerHandler handler, ITimePicker timePicker)

diff --git a/src/Core/src/Handlers/View/ViewHandlerOfT.cs b/src/Core/src/Handlers/View/ViewHandlerOfT.cs
@@ -24,12 +24,10 @@ public abstract partial class ViewHandler<TVirtualView, TNativeView> : ViewHandl
 	{
 		protected readonly PropertyMapper _defaultMapper;
 		protected PropertyMapper _mapper;
-		static bool HasSetDefaults;
 
 		[HotReload.OnHotReload]
 		static void OnHotReload()
 		{
-			HasSetDefaults = false;
 		}
 
 		protected ViewHandler(PropertyMapper mapper)
@@ -70,16 +68,6 @@ public override void SetVirtualView(IView view)
 				ConnectHandler(NativeView);
 			}
 
-			if (!HasSetDefaults)
-			{
-				if (NativeView != null)
-				{
-					SetupDefaults(NativeView);
-				}
-
-				HasSetDefaults = true;
-			}
-
 			_mapper = _defaultMapper;
 
 			if (VirtualView is IPropertyMapperView imv)
@@ -121,9 +109,6 @@ public override void UpdateValue(string property)
 			_mapper?.UpdateProperty(this, VirtualView, property);
 		}
 
-		protected virtual void SetupDefaults(TNativeView nativeView) { }
-
-
 		public new TVirtualView VirtualView
 		{
 			get => (TVirtualView?)base.VirtualView ?? throw new InvalidOperationException($"VirtualView cannot be null here");