Fix emoji popup

Stephen Paul Weber created 2 years ago

Change summary

src/cheogram/res/layout/emoji_search.xml                          |  1 
src/cheogram/res/layout/emoji_search_row.xml                      | 53 
src/main/java/eu/siacs/conversations/ui/ConversationFragment.java |  2 
3 files changed, 29 insertions(+), 27 deletions(-)

Detailed changes

src/cheogram/res/layout/emoji_search.xml

@@ -10,6 +10,7 @@
       android:layout_alignParentLeft="true"
       android:layout_alignParentTop="true"
       android:divider="@android:color/transparent"
+	android:background="?colorSurfaceVariant">
       android:dividerHeight="0dp"></ListView>
 
 </layout>

src/cheogram/res/layout/emoji_search_row.xml

@@ -3,38 +3,39 @@
     xmlns:app="http://schemas.android.com/apk/res-auto">
 
   <LinearLayout
-      android:layout_width="fill_parent"
-      android:layout_height="wrap_content"
-      android:layout_centerVertical="true"
-      android:orientation="horizontal"
-		android:paddingTop="8dp"
-		android:paddingBottom="8dp"
-		android:paddingLeft="@dimen/avatar_item_distance"
-		android:paddingRight="@dimen/avatar_item_distance">
+	android:layout_width="fill_parent"
+	android:layout_height="wrap_content"
+	android:layout_centerVertical="true"
+	android:orientation="horizontal"
+	android:paddingTop="8dp"
+	android:paddingBottom="8dp"
+	android:paddingLeft="@dimen/avatar_item_distance"
+	android:paddingRight="@dimen/avatar_item_distance"
+	android:background="@drawable/background_selectable_list_item">
 
 		<TextView
-				android:id="@+id/nonunicode"
-				android:layout_width="wrap_content"
-				android:layout_height="wrap_content"
-				android:gravity="center_vertical"
-				android:textAppearance="?textAppearanceBodyMedium"
-				app:emojiCompatEnabled="false"
-				android:visibility="gone" />
+			android:id="@+id/nonunicode"
+			android:layout_width="wrap_content"
+			android:layout_height="wrap_content"
+			android:gravity="center_vertical"
+			android:textAppearance="?textAppearanceBodyMedium"
+			app:emojiCompatEnabled="false"
+			android:visibility="gone" />
 
 		<TextView
-				android:id="@+id/unicode"
-				android:layout_width="wrap_content"
-				android:layout_height="wrap_content"
-				android:gravity="center_vertical"
-				android:textAppearance="?textAppearanceBodyMedium" />
+			android:id="@+id/unicode"
+			android:layout_width="wrap_content"
+			android:layout_height="wrap_content"
+			android:gravity="center_vertical"
+			android:textAppearance="?textAppearanceBodyMedium" />
 
 		<TextView
-				android:id="@+id/shortcode"
-				android:layout_width="wrap_content"
-				android:layout_height="wrap_content"
-				android:gravity="center_vertical"
-				android:paddingLeft="8dp"
-				android:textAppearance="?textAppearanceBodyMedium" />
+			android:id="@+id/shortcode"
+			android:layout_width="wrap_content"
+			android:layout_height="wrap_content"
+			android:gravity="center_vertical"
+			android:paddingLeft="8dp"
+			android:textAppearance="?textAppearanceBodyMedium" />
 
 	</LinearLayout>
 

src/main/java/eu/siacs/conversations/ui/ConversationFragment.java

@@ -1480,7 +1480,7 @@ public class ConversationFragment extends XmppFragment
                         EmojiSearch.EmojiSearchAdapter adapter = ((EmojiSearch.EmojiSearchAdapter) emojiSearchBinding.emoji.getAdapter());
                         if (adapter != null) {
                             adapter.search(q);
-                            emojiPopup.showAsDropDown(binding.textinput);
+                            emojiPopup.showAsDropDown(binding.textsend);
                         }
                     }
                 }, 400L);