From ca7695854ae02e8cf940ab1421dc0527d3caf498 Mon Sep 17 00:00:00 2001
From: Simon Chae <chaesmn@gmail.com>
Date: Mon, 11 May 2020 12:44:18 -0400
Subject: [PATCH] For #10552: Set initial add-on details fragment visibility to
 false (#10565)

---
 .../mozilla/fenix/addons/InstalledAddonDetailsFragment.kt | 8 ++++++--
 .../main/res/layout/fragment_installed_add_on_details.xml | 2 ++
 2 files changed, 8 insertions(+), 2 deletions(-)

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 3988daa8b..6ba5dbd10 100644
--- a/app/src/main/java/org/mozilla/fenix/addons/InstalledAddonDetailsFragment.kt
+++ b/app/src/main/java/org/mozilla/fenix/addons/InstalledAddonDetailsFragment.kt
@@ -14,6 +14,7 @@ import androidx.fragment.app.Fragment
 import androidx.lifecycle.lifecycleScope
 import androidx.navigation.Navigation
 import androidx.navigation.findNavController
+import androidx.navigation.fragment.findNavController
 import kotlinx.android.synthetic.main.fragment_installed_add_on_details.view.*
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.launch
@@ -41,7 +42,9 @@ class InstalledAddonDetailsFragment : Fragment() {
             addon = AddonDetailsFragmentArgs.fromBundle(requireNotNull(arguments)).addon
         }
 
-        return inflater.inflate(R.layout.fragment_installed_add_on_details, container, false)
+        return inflater.inflate(R.layout.fragment_installed_add_on_details, container, false).also {
+            bindUI(it)
+        }
     }
 
     override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
@@ -63,6 +66,7 @@ class InstalledAddonDetailsFragment : Fragment() {
                                 bindUI(view)
                             }
                             view.add_on_progress_bar.isVisible = false
+                            view.addon_container.isVisible = true
                         }
                     }
                 }
@@ -73,7 +77,7 @@ class InstalledAddonDetailsFragment : Fragment() {
                             view,
                             getString(R.string.mozac_feature_addons_failed_to_query_add_ons)
                         )
-                        view.add_on_progress_bar.isVisible = false
+                        findNavController().popBackStack()
                     }
                 }
             }
diff --git a/app/src/main/res/layout/fragment_installed_add_on_details.xml b/app/src/main/res/layout/fragment_installed_add_on_details.xml
index 32c9103a2..c143f53cc 100644
--- a/app/src/main/res/layout/fragment_installed_add_on_details.xml
+++ b/app/src/main/res/layout/fragment_installed_add_on_details.xml
@@ -13,8 +13,10 @@
         android:layout_height="wrap_content">
 
         <RelativeLayout
+            android:id="@+id/addon_container"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
+            android:visibility="gone"
             android:orientation="vertical">
 
             <Switch