From f12c2f16174ffec078b2fa2e23c607bf3a89b36f Mon Sep 17 00:00:00 2001
From: Colin Lee <mncolinlee@gmail.com>
Date: Tue, 2 Apr 2019 11:34:13 -0500
Subject: [PATCH] [fenix] No issue: Fix bookmark favicon background constraint
 ratio and alignment

---
 app/src/main/res/layout/bookmark_row.xml | 28 ++++++++++++------------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/app/src/main/res/layout/bookmark_row.xml b/app/src/main/res/layout/bookmark_row.xml
index 31805bfbd5..ff19b38cf5 100644
--- a/app/src/main/res/layout/bookmark_row.xml
+++ b/app/src/main/res/layout/bookmark_row.xml
@@ -15,42 +15,42 @@
 
     <ImageView
         android:id="@+id/bookmark_favicon"
-        android:layout_width="wrap_content"
+        android:layout_width="0dp"
         android:layout_height="wrap_content"
+        android:background="@drawable/favicon_background"
+        android:importantForAccessibility="no"
+        android:padding="10dp"
         app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintTop_toTopOf="parent"
-        app:layout_constraintBottom_toBottomOf="parent"
-        tools:src="@drawable/ic_folder_icon"
+        app:layout_constraintDimensionRatio="1:1"
         tools:foregroundTint="@android:color/black"
-        android:padding="10dp"
-        android:importantForAccessibility="no"
-        android:background="@drawable/favicon_background"/>
+        tools:src="@drawable/ic_folder_icon" />
 
     <TextView
         android:id="@+id/bookmark_title"
-        android:layout_width="wrap_content"
+        android:layout_width="0dp"
         android:layout_height="wrap_content"
         android:layout_marginStart="8dp"
         android:ellipsize="end"
         android:lines="1"
         android:textSize="16sp"
-        app:layout_constraintTop_toTopOf="parent"
         app:layout_constraintBottom_toBottomOf="parent"
-        app:layout_constraintStart_toEndOf="@id/bookmark_favicon"
         app:layout_constraintEnd_toStartOf="@id/bookmark_overflow"
         app:layout_constraintHorizontal_bias="0"
-        tools:text="Internet"/>
+        app:layout_constraintStart_toEndOf="@id/bookmark_favicon"
+        app:layout_constraintTop_toTopOf="parent"
+        tools:text="Internet" />
 
     <ImageView
         android:id="@+id/bookmark_overflow"
         android:layout_width="24dp"
-        android:layout_height="24dp"
-        android:src="@drawable/ic_menu"
+        android:layout_height="wrap_content"
         android:layout_margin="10dp"
-        app:layout_constraintTop_toTopOf="parent"
+        android:contentDescription="@string/bookmark_menu_content_description"
+        android:src="@drawable/ic_menu"
         app:layout_constraintBottom_toBottomOf="parent"
         app:layout_constraintEnd_toEndOf="parent"
-        android:contentDescription="@string/bookmark_menu_content_description"/>
+        app:layout_constraintTop_toTopOf="parent" />
     
     <View
         android:id="@+id/bookmark_separator"