Sets visibility to gone when not a milestone to prevent hiding other …

…notification types
wordpress-mobile · Mar 22, 2024 · 2a7e26f · 2a7e26f
1 parent c10060b
commit 2a7e26f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/...s/src/main/java/org/wordpress/android/ui/notifications/NotificationsDetailListFragment.kt b/...s/src/main/java/org/wordpress/android/ui/notifications/NotificationsDetailListFragment.kt
@@ -133,8 +133,12 @@ class NotificationsDetailListFragment : ListFragment(), NotificationFragment {
             showErrorToastAndFinish()
         }
 
+        val animation = view?.findViewById<LottieAnimationView>(R.id.confetti)
         if (note?.isViewMilestoneType == true) {
-            view?.findViewById<LottieAnimationView>(R.id.confetti)?.playAnimation()
+            animation?.visibility = View.VISIBLE
+            animation?.playAnimation()
+        } else {
+            animation?.visibility = View.GONE
         }
     }
 

diff --git a/WordPress/src/main/res/layout/notifications_fragment_detail_list.xml b/WordPress/src/main/res/layout/notifications_fragment_detail_list.xml
@@ -12,6 +12,7 @@
         android:layout_height="match_parent"
         android:layout_gravity="top"
         android:scaleType="centerCrop"
+        android:visibility="gone"
         app:lottie_autoPlay="false"
         app:lottie_loop="false"
         app:lottie_rawRes="@raw/confetti" />