From 2f01ad99b375a8d2f1011d6ece6e3c7ebf4a007d Mon Sep 17 00:00:00 2001
From: Valere <valeref@matrix.org>
Date: Tue, 16 Jul 2019 16:35:36 +0200
Subject: [PATCH] Compact long tap menu

---
 .../adapter_item_action_quick_reaction.xml     | 18 ++----------------
 .../layout/bottom_sheet_message_actions.xml    |  1 +
 vector/src/main/res/values/strings.xml         |  1 -
 3 files changed, 3 insertions(+), 17 deletions(-)

diff --git a/vector/src/main/res/layout/adapter_item_action_quick_reaction.xml b/vector/src/main/res/layout/adapter_item_action_quick_reaction.xml
index 9f26e95f..16a53cff 100644
--- a/vector/src/main/res/layout/adapter_item_action_quick_reaction.xml
+++ b/vector/src/main/res/layout/adapter_item_action_quick_reaction.xml
@@ -7,18 +7,6 @@
     android:layout_height="wrap_content"
     android:padding="8dp">
 
-    <TextView
-        android:id="@+id/quickReactionTitle"
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:layout_marginStart="8dp"
-        android:layout_marginEnd="8dp"
-        android:text="@string/quick_reactions"
-        android:textColor="?riotx_text_secondary"
-        android:textSize="16sp"
-        app:layout_constraintTop_toTopOf="parent" />
-
-
     <TextView
         android:id="@+id/quickReaction0"
         android:layout_width="wrap_content"
@@ -104,16 +92,14 @@
         android:id="@+id/reactionsFlowHelper"
         android:layout_width="0dp"
         android:layout_height="wrap_content"
-        android:layout_marginTop="8dp"
-        android:layout_marginBottom="16dp"
         app:constraint_referenced_ids="quickReaction0,quickReaction1,quickReaction2,quickReaction3,quickReaction4,quickReaction5,quickReaction6,quickReaction7"
-        app:flow_horizontalGap="8dp"
+        app:flow_horizontalGap="0dp"
         app:flow_horizontalStyle="spread"
         app:flow_verticalBias="0"
         app:flow_verticalGap="4dp"
         app:flow_wrapMode="chain"
         app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintStart_toStartOf="parent"
-        app:layout_constraintTop_toBottomOf="@id/quickReactionTitle" />
+        app:layout_constraintTop_toTopOf="parent" />
 
 </androidx.constraintlayout.widget.ConstraintLayout>
diff --git a/vector/src/main/res/layout/bottom_sheet_message_actions.xml b/vector/src/main/res/layout/bottom_sheet_message_actions.xml
index d1cb8c9f..9fadcee1 100644
--- a/vector/src/main/res/layout/bottom_sheet_message_actions.xml
+++ b/vector/src/main/res/layout/bottom_sheet_message_actions.xml
@@ -28,6 +28,7 @@
                 app:layout_constraintBottom_toBottomOf="parent"
                 app:layout_constraintStart_toStartOf="parent"
                 app:layout_constraintTop_toTopOf="parent"
+                app:layout_constraintVertical_bias="0"
                 tools:src="@tools:sample/avatars" />
 
             <TextView
diff --git a/vector/src/main/res/values/strings.xml b/vector/src/main/res/values/strings.xml
index 96404fe6..71672eb2 100644
--- a/vector/src/main/res/values/strings.xml
+++ b/vector/src/main/res/values/strings.xml
@@ -1534,7 +1534,6 @@ Why choose Riot.im?
     <string name="settings_other_third_party_notices">Other third party notices</string>
     <string name="navigate_to_room_when_already_in_the_room">You are already viewing this room!</string>
 
-    <string name="quick_reactions">Quick Reactions</string>
 
     <!-- Settings -->
     <string name="settings_general_title">General</string>