Added a couple of minor UI optimisations

app/src/main/java/org/thoughtcrime/securesms/conversation/v2/messages/LinkPreviewView.kt

@@ -80,7 +80,7 @@ class LinkPreviewView : LinearLayout {
         val rawYInt = event.rawY.toInt()
         val hitRect = Rect(rawXInt, rawYInt, rawXInt, rawYInt)
         val previewRect = Rect()
-        binding.mainLinkPreviewParent.getGlobalVisibleRect(previewRect)
+        binding.mainLinkPreviewContainer.getGlobalVisibleRect(previewRect)
         if (previewRect.contains(hitRect)) {
             openURL()
             return

app/src/main/java/org/thoughtcrime/securesms/conversation/v2/messages/VisibleMessageView.kt

@@ -192,15 +192,16 @@ class VisibleMessageView : LinearLayout {
         binding.dateBreakTextView.text = if (showDateBreak) DateUtils.getDisplayFormattedTimeSpanString(context, Locale.getDefault(), message.timestamp) else null
         binding.dateBreakTextView.isVisible = showDateBreak
         // Message status indicator
-        val (iconID, iconColor) = getMessageStatusImage(message)
-        if (iconID != null) {
-            val drawable = ContextCompat.getDrawable(context, iconID)?.mutate()
-            if (iconColor != null) {
-                drawable?.setTint(iconColor)
-            }
-            binding.messageStatusImageView.setImageDrawable(drawable)
-        }
         if (message.isOutgoing) {
+            val (iconID, iconColor) = getMessageStatusImage(message)
+            if (iconID != null) {
+                val drawable = ContextCompat.getDrawable(context, iconID)?.mutate()
+                if (iconColor != null) {
+                    drawable?.setTint(iconColor)
+                }
+                binding.messageStatusImageView.setImageDrawable(drawable)
+            }
+
             val lastMessageID = mmsSmsDb.getLastMessageID(message.threadId)
             binding.messageStatusImageView.isVisible =
                 !message.isSent || message.id == lastMessageID
@@ -213,13 +214,17 @@ class VisibleMessageView : LinearLayout {
         val emojiLayoutParams = binding.emojiReactionsView.layoutParams as ConstraintLayout.LayoutParams
         emojiLayoutParams.horizontalBias = if (message.isOutgoing) 1f else 0f
         binding.emojiReactionsView.layoutParams = emojiLayoutParams
-        val capabilities = lokiThreadDb.getOpenGroupChat(threadID)?.server?.let { lokiApiDb.getServerCapabilities(it) }
+
-        if (message.reactions.isNotEmpty() &&
+        if (message.reactions.isNotEmpty()) {
-            (capabilities.isNullOrEmpty() || capabilities.contains(OpenGroupApi.Capability.REACTIONS.name.lowercase()))
+            val capabilities = lokiThreadDb.getOpenGroupChat(threadID)?.server?.let { lokiApiDb.getServerCapabilities(it) }
-        ) {
+            if (capabilities.isNullOrEmpty() || capabilities.contains(OpenGroupApi.Capability.REACTIONS.name.lowercase())) {
-            binding.emojiReactionsView.setReactions(message.id, message.reactions, message.isOutgoing, delegate)
+                binding.emojiReactionsView.setReactions(message.id, message.reactions, message.isOutgoing, delegate)
-            binding.emojiReactionsView.isVisible = true
+                binding.emojiReactionsView.isVisible = true
-        } else {
+            } else {
+                binding.emojiReactionsView.isVisible = false
+            }
+        }
+        else {
             binding.emojiReactionsView.isVisible = false
         }
 

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

@@ -1,54 +1,46 @@
 <?xml version="1.0" encoding="utf-8"?>
 <LinearLayout android:id="@+id/mainLinkPreviewContainer"
     xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:tools="http://schemas.android.com/tools"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    android:background="@color/transparent_black_6"
     android:layout_width="300dp"
     android:layout_height="wrap_content"
-    xmlns:tools="http://schemas.android.com/tools"
+    android:orientation="horizontal"
-    android:orientation="vertical"
+    android:gravity="center">
-    xmlns:app="http://schemas.android.com/apk/res-auto">
 
-    <LinearLayout
+    <RelativeLayout
-        android:background="@color/transparent_black_6"
+        android:layout_width="96dp"
-        android:id="@+id/mainLinkPreviewParent"
+        android:layout_height="96dp">
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:orientation="horizontal"
-        android:gravity="center">
 
-        <RelativeLayout
+        <ImageView
-            android:layout_width="96dp"
+            android:layout_width="24dp"
-            android:layout_height="96dp">
+            android:layout_height="24dp"
+            android:layout_centerInParent="true"
+            android:src="@drawable/ic_link"
+            app:tint="?android:textColorPrimary" />
 
-            <ImageView
+        <org.thoughtcrime.securesms.conversation.v2.utilities.KThumbnailView
-                android:layout_width="24dp"
+            android:background="@color/transparent_black_6"
-                android:layout_height="24dp"
+            android:id="@+id/thumbnailImageView"
-                android:layout_centerInParent="true"
-                android:src="@drawable/ic_link"
-                app:tint="?android:textColorPrimary" />
-
-            <org.thoughtcrime.securesms.conversation.v2.utilities.KThumbnailView
-                android:background="@color/transparent_black_6"
-                android:id="@+id/thumbnailImageView"
-                android:layout_width="match_parent"
-                android:layout_height="match_parent"
-                android:scaleType="centerCrop" />
-
-        </RelativeLayout>
-
-        <TextView
-            android:id="@+id/titleTextView"
             android:layout_width="match_parent"
             android:layout_height="match_parent"
-            android:paddingHorizontal="12dp"
+            android:scaleType="centerCrop" />
-            android:gravity="center_vertical"
-            android:textSize="@dimen/small_font_size"
-            android:textStyle="bold"
-            tools:text="Some Text here"
-            android:minWidth="@dimen/media_bubble_min_width"
-            android:maxLines="3"
-            android:ellipsize="end"
-            android:textColor="?android:textColorPrimary"/>
 
-    </LinearLayout>
+    </RelativeLayout>
+
+    <TextView
+        android:id="@+id/titleTextView"
+        android:layout_width="match_parent"
+        android:layout_height="match_parent"
+        android:paddingHorizontal="12dp"
+        android:gravity="center_vertical"
+        android:textSize="@dimen/small_font_size"
+        android:textStyle="bold"
+        tools:text="Some Text here"
+        android:minWidth="@dimen/media_bubble_min_width"
+        android:maxLines="3"
+        android:ellipsize="end"
+        android:textColor="?android:textColorPrimary"/>
 
 </LinearLayout>