From 682176474cd052293d866fa7e254e4d335f6b954 Mon Sep 17 00:00:00 2001 From: Simon Chae Date: Thu, 23 Apr 2020 16:25:03 -0400 Subject: [PATCH] For #10162: Set switch back to prev state when failed to enable/disable addon --- .../org/mozilla/fenix/addons/InstalledAddonDetailsFragment.kt | 3 +++ 1 file changed, 3 insertions(+) diff --git a/app/src/main/java/org/mozilla/fenix/addons/InstalledAddonDetailsFragment.kt b/app/src/main/java/org/mozilla/fenix/addons/InstalledAddonDetailsFragment.kt index df5a40e80c71..5793aaca818c 100644 --- a/app/src/main/java/org/mozilla/fenix/addons/InstalledAddonDetailsFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/addons/InstalledAddonDetailsFragment.kt @@ -116,6 +116,7 @@ class InstalledAddonDetailsFragment : Fragment() { runIfFragmentIsAttached { switch.isClickable = true view.remove_add_on.isEnabled = true + switch.setState(addon.isEnabled()) showSnackBar( view, getString( @@ -149,6 +150,7 @@ class InstalledAddonDetailsFragment : Fragment() { runIfFragmentIsAttached { switch.isClickable = true view.remove_add_on.isEnabled = true + switch.setState(addon.isEnabled()) showSnackBar( view, getString( @@ -230,6 +232,7 @@ class InstalledAddonDetailsFragment : Fragment() { } }, onError = { + switch.isChecked = addon.isAllowedInPrivateBrowsing() runIfFragmentIsAttached { switch.isClickable = true view.remove_add_on.isEnabled = true