Do not allow BadgeImageView to control its own visibility.

app/src/main/java/org/thoughtcrime/securesms/badges/BadgeImageView.kt

@@ -31,7 +31,7 @@ class BadgeImageView @JvmOverloads constructor(
   fun setBadgeFromRecipient(recipient: Recipient?) {
     getGlideRequests()?.let {
       setBadgeFromRecipient(recipient, it)
-    }
+    } ?: setImageDrawable(null)
   }
 
   fun setBadgeFromRecipient(recipient: Recipient?, glideRequests: GlideRequests) {
@@ -45,12 +45,10 @@ class BadgeImageView @JvmOverloads constructor(
   fun setBadge(badge: Badge?) {
     getGlideRequests()?.let {
       setBadge(badge, it)
-    }
+    } ?: setImageDrawable(null)
   }
 
   fun setBadge(badge: Badge?, glideRequests: GlideRequests) {
-    visible = badge != null
-
     if (badge != null) {
       glideRequests
         .load(badge)
@@ -60,6 +58,7 @@ class BadgeImageView @JvmOverloads constructor(
     } else {
       glideRequests
         .clear(this)
+      setImageDrawable(null)
     }
   }
 

app/src/main/res/layout/bio_preference_item.xml

@@ -26,11 +26,9 @@
         android:layout_marginStart="40dp"
         android:layout_marginTop="40dp"
         android:contentDescription="@string/ImageView__badge"
-        android:visibility="gone"
         app:badge_size="medium"
         app:layout_constraintStart_toStartOf="@id/icon"
-        app:layout_constraintTop_toTopOf="@id/icon"
+        app:layout_constraintTop_toTopOf="@id/icon" />
-        tools:visibility="visible" />
 
     <LinearLayout
         android:layout_width="0dp"

app/src/main/res/layout/contact_selection_list_item.xml

@@ -30,7 +30,6 @@
         android:layout_marginStart="20dp"
         android:layout_marginTop="22dp"
         android:contentDescription="@string/ImageView__badge"
-        android:visibility="gone"
         app:badge_size="medium"
         app:layout_constraintStart_toStartOf="@id/contact_photo_image"
         app:layout_constraintTop_toTopOf="@id/contact_photo_image"

app/src/main/res/layout/conversation_banner_view.xml

@@ -23,11 +23,9 @@
         android:layout_marginStart="47dp"
         android:layout_marginTop="48dp"
         android:contentDescription="@string/ImageView__badge"
-        android:visibility="gone"
         app:badge_size="large"
         app:layout_constraintStart_toStartOf="@id/message_request_avatar"
-        app:layout_constraintTop_toTopOf="@id/message_request_avatar"
+        app:layout_constraintTop_toTopOf="@id/message_request_avatar" />
-        tools:visibility="visible" />
 
     <LinearLayout
         android:id="@+id/message_request_avatar_tap_to_view"

app/src/main/res/layout/conversation_item_received_multimedia.xml

@@ -62,8 +62,7 @@
         android:layout_marginStart="14dp"
         android:layout_marginTop="16dp"
         android:visibility="gone"
-        app:badge_size="small"
+        app:badge_size="small" />
-        tools:background="@color/red" />
 
     <org.thoughtcrime.securesms.conversation.ConversationItemBodyBubble
         android:id="@+id/body_bubble"

app/src/main/res/layout/conversation_item_received_text_only.xml

@@ -62,8 +62,7 @@
         android:layout_marginStart="14dp"
         android:layout_marginTop="16dp"
         android:visibility="gone"
-        app:badge_size="small"
+        app:badge_size="small" />
-        tools:background="@color/red" />
 
     <org.thoughtcrime.securesms.conversation.ConversationItemBodyBubble
         android:id="@+id/body_bubble"

app/src/main/res/layout/conversation_list_item_view.xml

@@ -181,10 +181,8 @@
         android:layout_marginStart="26dp"
         android:layout_marginTop="26dp"
         android:contentDescription="@string/ImageView__badge"
-        android:visibility="gone"
         app:badge_size="medium"
         app:layout_constraintStart_toStartOf="@id/conversation_list_item_avatar"
-        app:layout_constraintTop_toTopOf="@id/conversation_list_item_avatar"
+        app:layout_constraintTop_toTopOf="@id/conversation_list_item_avatar" />
-        tools:visibility="visible" />
 
 </org.thoughtcrime.securesms.conversationlist.ConversationListItem>

app/src/main/res/layout/conversation_settings_avatar_preference_item.xml

@@ -28,10 +28,8 @@
             android:layout_marginStart="47dp"
             android:layout_marginTop="48dp"
             android:contentDescription="@string/ImageView__badge"
-            android:visibility="gone"
             app:badge_size="large"
             app:layout_constraintStart_toStartOf="@id/bio_preference_avatar"
-            app:layout_constraintTop_toTopOf="@id/bio_preference_avatar"
+            app:layout_constraintTop_toTopOf="@id/bio_preference_avatar" />
-            tools:visibility="visible" />
     </androidx.constraintlayout.widget.ConstraintLayout>
 </FrameLayout>

app/src/main/res/layout/conversation_title_view.xml

@@ -38,9 +38,7 @@
             android:layout_marginStart="22dp"
             android:layout_marginTop="22dp"
             android:contentDescription="@string/ImageView__badge"
-            android:visibility="gone"
+            app:badge_size="small" />
-            app:badge_size="small"
-            tools:visibility="visible" />
     </RelativeLayout>
 
     <LinearLayout

app/src/main/res/layout/group_recipient_list_item.xml

@@ -26,11 +26,9 @@
         android:layout_marginStart="20dp"
         android:layout_marginTop="22dp"
         android:contentDescription="@string/ImageView__badge"
-        android:visibility="gone"
         app:badge_size="medium"
         app:layout_constraintStart_toStartOf="@id/recipient_avatar"
-        app:layout_constraintTop_toTopOf="@id/recipient_avatar"
+        app:layout_constraintTop_toTopOf="@id/recipient_avatar" />
-        tools:visibility="visible" />
 
     <androidx.appcompat.widget.AppCompatCheckBox
         android:id="@+id/recipient_selected"

app/src/main/res/layout/message_details_recipient.xml

@@ -37,11 +37,9 @@
             android:layout_marginStart="20dp"
             android:layout_marginTop="22dp"
             android:contentDescription="@string/ImageView__badge"
-            android:visibility="gone"
             app:badge_size="medium"
             app:layout_constraintStart_toStartOf="@id/message_details_recipient_avatar"
-            app:layout_constraintTop_toTopOf="@id/message_details_recipient_avatar"
+            app:layout_constraintTop_toTopOf="@id/message_details_recipient_avatar" />
-            tools:visibility="visible" />
 
         <org.thoughtcrime.securesms.components.FromTextView
             android:id="@+id/message_details_recipient_name"

app/src/main/res/layout/recipient_bottom_sheet.xml

@@ -28,11 +28,9 @@
             android:layout_marginStart="47dp"
             android:layout_marginTop="48dp"
             android:contentDescription="@string/ImageView__badge"
-            android:visibility="gone"
             app:badge_size="large"
             app:layout_constraintStart_toStartOf="@id/rbs_recipient_avatar"
-            app:layout_constraintTop_toTopOf="@id/rbs_recipient_avatar"
+            app:layout_constraintTop_toTopOf="@id/rbs_recipient_avatar" />
-            tools:visibility="visible" />
 
         <ProgressBar
             android:id="@+id/rbs_admin_action_busy"