diff options
| author | E.Z. Hart <hartez@users.noreply.github.com> | 2017-06-23 18:29:38 -0600 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2017-06-23 18:29:38 -0600 |
| commit | eb2a2177b3761efbc9b6563d4621ab4a19300837 (patch) | |
| tree | 7f6fc3bba203bdcf33059a283cb8d9e37051fd2d /Xamarin.Forms.Platform.Android | |
| parent | 4847e4b26a0b7808b6af47854946ee10fc46adc2 (diff) | |
| download | xamarin-forms-eb2a2177b3761efbc9b6563d4621ab4a19300837.tar.gz xamarin-forms-eb2a2177b3761efbc9b6563d4621ab4a19300837.tar.bz2 xamarin-forms-eb2a2177b3761efbc9b6563d4621ab4a19300837.zip | |
Fix NRE when background color of button set in FormsApplicationActivity (#1010)
Diffstat (limited to 'Xamarin.Forms.Platform.Android')
| -rw-r--r-- | Xamarin.Forms.Platform.Android/AppCompat/ButtonRenderer.cs | 2 | ||||
| -rw-r--r-- | Xamarin.Forms.Platform.Android/Renderers/ButtonRenderer.cs | 12 |
2 files changed, 8 insertions, 6 deletions
diff --git a/Xamarin.Forms.Platform.Android/AppCompat/ButtonRenderer.cs b/Xamarin.Forms.Platform.Android/AppCompat/ButtonRenderer.cs index e7e9a3bf..9f336a24 100644 --- a/Xamarin.Forms.Platform.Android/AppCompat/ButtonRenderer.cs +++ b/Xamarin.Forms.Platform.Android/AppCompat/ButtonRenderer.cs @@ -140,7 +140,7 @@ namespace Xamarin.Forms.Platform.Android.AppCompat if (Element == null || Control == null) return; - _backgroundTracker.UpdateBackgroundColor(); + _backgroundTracker?.UpdateBackgroundColor(); } void UpdateAll() diff --git a/Xamarin.Forms.Platform.Android/Renderers/ButtonRenderer.cs b/Xamarin.Forms.Platform.Android/Renderers/ButtonRenderer.cs index da4ec0ba..0aac4edb 100644 --- a/Xamarin.Forms.Platform.Android/Renderers/ButtonRenderer.cs +++ b/Xamarin.Forms.Platform.Android/Renderers/ButtonRenderer.cs @@ -96,13 +96,12 @@ namespace Xamarin.Forms.Platform.Android _textColorSwitcher = new TextColorSwitcher(button.TextColors); button.AddOnAttachStateChangeListener(this); } - if (_backgroundTracker == null) - _backgroundTracker = new ButtonBackgroundTracker(Element, Control); } + + if (_backgroundTracker == null) + _backgroundTracker = new ButtonBackgroundTracker(Element, Control); else - { _backgroundTracker.Button = e.NewElement; - } UpdateAll(); } @@ -127,7 +126,10 @@ namespace Xamarin.Forms.Platform.Android protected override void UpdateBackgroundColor() { - _backgroundTracker.UpdateBackgroundColor(); + if (Element == null || Control == null) + return; + + _backgroundTracker?.UpdateBackgroundColor(); } void UpdateAll() |