From 9e12fd7e39ec2ecdaf16ec8d82cf34a5f0653a1b Mon Sep 17 00:00:00 2001 From: Cody Henthorne Date: Wed, 29 Jul 2026 20:37:36 -0400 Subject: [PATCH] Fix story quote author name truncation. --- .../securesms/components/QuoteView.java | 2 +- .../V2ConversationItemMediaViewHolder.kt | 19 +++++++------- app/src/main/res/layout/v2_quote_view.xml | 26 +++++++++++++++---- 3 files changed, 32 insertions(+), 15 deletions(-) diff --git a/app/src/main/java/org/thoughtcrime/securesms/components/QuoteView.java b/app/src/main/java/org/thoughtcrime/securesms/components/QuoteView.java index 63ef7cbf71..b64d40582c 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/components/QuoteView.java +++ b/app/src/main/java/org/thoughtcrime/securesms/components/QuoteView.java @@ -322,7 +322,7 @@ public class QuoteView extends ConstraintLayout implements RecipientForeverObser if (storyReaction != null) { storyReactionEmoji.setImageEmoji(storyReaction); storyReactionEmoji.setVisibility(View.VISIBLE); - missingStoryReaction.setVisibility(View.INVISIBLE); + missingStoryReaction.setVisibility(View.GONE); } else { storyReactionEmoji.setVisibility(View.GONE); missingStoryReaction.setVisibility(View.GONE); diff --git a/app/src/main/java/org/thoughtcrime/securesms/conversation/v2/items/V2ConversationItemMediaViewHolder.kt b/app/src/main/java/org/thoughtcrime/securesms/conversation/v2/items/V2ConversationItemMediaViewHolder.kt index 96dc940b16..88bbc63240 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/conversation/v2/items/V2ConversationItemMediaViewHolder.kt +++ b/app/src/main/java/org/thoughtcrime/securesms/conversation/v2/items/V2ConversationItemMediaViewHolder.kt @@ -80,6 +80,16 @@ class V2ConversationItemMediaViewHolder>( } binding.quoteStub.visibility = View.VISIBLE + + quoteView.setMessageType( + when { + conversationMessage.messageRecord.isStoryReaction() && binding.textBridge.isIncoming -> QuoteView.MessageType.STORY_REPLY_INCOMING + conversationMessage.messageRecord.isStoryReaction() -> QuoteView.MessageType.STORY_REPLY_OUTGOING + binding.textBridge.isIncoming -> QuoteView.MessageType.INCOMING + else -> QuoteView.MessageType.OUTGOING + } + ) + quoteView.setQuote( conversationContext.requestManager, quote.id, @@ -93,15 +103,6 @@ class V2ConversationItemMediaViewHolder>( conversationMessage.quoteMemberLabel ) - quoteView.setMessageType( - when { - conversationMessage.messageRecord.isStoryReaction() && binding.textBridge.isIncoming -> QuoteView.MessageType.STORY_REPLY_INCOMING - conversationMessage.messageRecord.isStoryReaction() -> QuoteView.MessageType.STORY_REPLY_OUTGOING - binding.textBridge.isIncoming -> QuoteView.MessageType.INCOMING - else -> QuoteView.MessageType.OUTGOING - } - ) - quoteView.setWallpaperEnabled(conversationContext.hasWallpaper()) quoteView.setTextSize(TypedValue.COMPLEX_UNIT_SP, SignalStore.settings.getMessageQuoteFontSize(context).toFloat()) diff --git a/app/src/main/res/layout/v2_quote_view.xml b/app/src/main/res/layout/v2_quote_view.xml index bd34df381c..a57296eac6 100644 --- a/app/src/main/res/layout/v2_quote_view.xml +++ b/app/src/main/res/layout/v2_quote_view.xml @@ -25,11 +25,11 @@ android:layout_height="wrap_content" android:layout_marginStart="8dp" android:layout_marginTop="8dp" - android:layout_marginEnd="28dp" + android:layout_marginEnd="8dp" android:layout_marginBottom="1dp" app:layout_constrainedWidth="true" app:layout_constraintBottom_toTopOf="@id/quote_attachment_name_stub" - app:layout_constraintEnd_toStartOf="@id/quote_missing_story_reaction_emoji" + app:layout_constraintEnd_toStartOf="@id/quote_author_end_barrier" app:layout_constraintHorizontal_bias="0" app:layout_constraintStart_toEndOf="@id/quote_bar" app:layout_constraintTop_toTopOf="parent" @@ -52,7 +52,7 @@ android:inflatedId="@id/quote_attachment_name_stub" android:layout="@layout/v2_quote_view_attachment_name_stub" app:layout_constraintBottom_toTopOf="@id/media_type" - app:layout_constraintEnd_toStartOf="@id/quote_missing_story_reaction_emoji" + app:layout_constraintEnd_toStartOf="@id/quote_body_end_barrier" app:layout_constraintStart_toEndOf="@id/quote_bar" app:layout_constraintTop_toBottomOf="@id/quote_author_holder" app:layout_constraintWidth_default="spread" @@ -69,7 +69,7 @@ android:textColor="@color/core_black" android:visibility="gone" app:layout_constraintBottom_toTopOf="@id/quote_text" - app:layout_constraintEnd_toStartOf="@id/quote_missing_story_reaction_emoji" + app:layout_constraintEnd_toStartOf="@id/quote_body_end_barrier" app:layout_constraintHorizontal_bias="0" app:layout_constraintStart_toEndOf="@id/quote_bar" app:layout_constraintTop_toBottomOf="@id/quote_attachment_name_stub" @@ -91,7 +91,7 @@ app:emoji_renderSpoilers="true" app:layout_constrainedWidth="true" app:layout_constraintBottom_toTopOf="@id/content_barrier" - app:layout_constraintEnd_toStartOf="@id/quote_missing_story_reaction_emoji" + app:layout_constraintEnd_toStartOf="@id/quote_body_end_barrier" app:layout_constraintHorizontal_bias="0" app:layout_constraintStart_toEndOf="@id/quote_bar" app:layout_constraintTop_toBottomOf="@id/media_type" @@ -144,6 +144,22 @@ app:layout_constraintEnd_toEndOf="parent" app:layout_constraintTop_toTopOf="parent" /> + + + + + +