From 2eaadd43376f19f50d3193bfd58e49ff27586673 Mon Sep 17 00:00:00 2001
From: Greyson Parrelli <greyson@signal.org>
Date: Fri, 10 Sep 2021 10:47:32 -0400
Subject: [PATCH] Allow multi-line text in media send flow.

---
 .../rounded_rectangle_secondary_dark.xml       |  2 +-
 .../v2_media_add_message_dialog_fragment.xml   | 18 +++++++++++-------
 .../res/layout/v2_media_review_fragment.xml    |  9 ++++++++-
 3 files changed, 20 insertions(+), 9 deletions(-)

diff --git a/app/src/main/res/drawable/rounded_rectangle_secondary_dark.xml b/app/src/main/res/drawable/rounded_rectangle_secondary_dark.xml
index 9772cf2b7..955aa219d 100644
--- a/app/src/main/res/drawable/rounded_rectangle_secondary_dark.xml
+++ b/app/src/main/res/drawable/rounded_rectangle_secondary_dark.xml
@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
 <shape xmlns:android="http://schemas.android.com/apk/res/android" android:shape="rectangle" >
     <solid android:color="@color/core_grey_75"/>
-    <corners android:radius="30dp"/>
+    <corners android:radius="20dp"/>
 </shape>
\ No newline at end of file
diff --git a/app/src/main/res/layout/v2_media_add_message_dialog_fragment.xml b/app/src/main/res/layout/v2_media_add_message_dialog_fragment.xml
index 20b1f3ca8..3e8c9ca07 100644
--- a/app/src/main/res/layout/v2_media_add_message_dialog_fragment.xml
+++ b/app/src/main/res/layout/v2_media_add_message_dialog_fragment.xml
@@ -39,13 +39,13 @@
             android:layout_height="@dimen/conversation_compose_height"
             android:layout_gravity="bottom"
             android:layout_marginStart="16dp"
+            android:layout_marginBottom="6dp"
             android:background="?selectableItemBackgroundBorderless"
             android:contentDescription="@string/conversation_activity__emoji_toggle_description"
-            android:paddingStart="4dp"
+            android:paddingStart="8dp"
             android:paddingEnd="6dp"
-            app:layout_constraintBottom_toBottomOf="@id/add_a_message_input"
-            app:layout_constraintStart_toStartOf="parent"
-            app:layout_constraintTop_toTopOf="@id/add_a_message_input" />
+            app:layout_constraintBottom_toTopOf="@id/emoji_drawer_stub"
+            app:layout_constraintStart_toStartOf="parent" />
 
         <org.thoughtcrime.securesms.components.ComposeText
             android:id="@+id/add_a_message_input"
@@ -56,10 +56,13 @@
             android:layout_marginBottom="8dp"
             android:background="@null"
             android:hint="@string/MediaReviewFragment__add_a_message"
-            android:inputType="textCapSentences"
+            android:inputType="textAutoCorrect|textCapSentences|textMultiLine"
             android:minHeight="36dp"
             android:paddingEnd="10dp"
-            android:textAppearance="@style/TextAppearance.Signal.Body2"
+            android:maxLines="3"
+            android:maxLength="65536"
+            android:imeOptions="flagNoEnterAction"
+            android:textAppearance="@style/Signal.Text.Body"
             app:layout_constraintBottom_toTopOf="@id/emoji_drawer_stub"
             app:layout_constraintEnd_toStartOf="@id/confirm_button"
             app:layout_constraintStart_toEndOf="@id/emoji_toggle"
@@ -71,9 +74,10 @@
             android:layout_width="48dp"
             android:layout_height="48dp"
             android:layout_marginEnd="8dp"
+            android:layout_marginBottom="2dp"
             android:padding="6dp"
             app:layout_constraintEnd_toEndOf="parent"
-            app:layout_constraintTop_toTopOf="@id/input_barrier"
+            app:layout_constraintBottom_toTopOf="@id/emoji_drawer_stub"
             app:srcCompat="@drawable/v2_media_add_a_message_check" />
 
         <ViewStub
diff --git a/app/src/main/res/layout/v2_media_review_fragment.xml b/app/src/main/res/layout/v2_media_review_fragment.xml
index beeaa802c..db20e15f0 100644
--- a/app/src/main/res/layout/v2_media_review_fragment.xml
+++ b/app/src/main/res/layout/v2_media_review_fragment.xml
@@ -151,7 +151,7 @@
             app:layout_constraintBottom_toTopOf="@id/button_barrier"
             tools:alpha="1" />
 
-        <org.thoughtcrime.securesms.components.emoji.EmojiTextView
+        <org.thoughtcrime.securesms.components.emoji.EmojiEditText
             android:id="@+id/add_a_message_entry"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
@@ -164,9 +164,16 @@
             android:minHeight="36dp"
             android:paddingStart="12dp"
             android:paddingEnd="12dp"
+            android:paddingTop="6dp"
+            android:paddingBottom="6dp"
             android:textAppearance="@style/Signal.Text.Body"
             android:textColor="@color/core_white"
             android:visibility="gone"
+            android:inputType="none"
+            android:enabled="false"
+            android:focusable="false"
+            android:cursorVisible="false"
+            android:maxLines="3"
             app:layout_constraintBottom_toTopOf="@id/button_barrier"
             app:layout_constraintTop_toTopOf="@id/add_a_message_barrier"
             tools:visibility="visible" />