From 15f09bd52a6e4bb7dd6a3269fc1b3c90b9b89806 Mon Sep 17 00:00:00 2001
From: tibbi <tibor@kaputa.sk>
Date: Thu, 27 Sep 2018 13:42:07 +0200
Subject: [PATCH] increase list item heights if phone number is shown

---
 app/src/main/res/layout/item_contact_with_number.xml | 9 +++++----
 app/src/main/res/layout/item_recent_call.xml         | 7 ++++---
 app/src/main/res/values/dimens.xml                   | 1 +
 3 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/app/src/main/res/layout/item_contact_with_number.xml b/app/src/main/res/layout/item_contact_with_number.xml
index e08b13ee..4ecf2645 100644
--- a/app/src/main/res/layout/item_contact_with_number.xml
+++ b/app/src/main/res/layout/item_contact_with_number.xml
@@ -13,13 +13,14 @@
     <RelativeLayout
         android:id="@+id/contact_holder"
         android:layout_width="match_parent"
-        android:layout_height="@dimen/contact_item_height"
+        android:layout_height="@dimen/contact_item_with_number_height"
+        android:paddingTop="@dimen/tiny_margin"
         android:paddingRight="@dimen/activity_margin">
 
         <ImageView
             android:id="@+id/contact_tmb"
             android:layout_width="@dimen/normal_icon_size"
-            android:layout_height="@dimen/normal_icon_size"
+            android:layout_height="match_parent"
             android:padding="@dimen/medium_margin"
             android:src="@drawable/ic_person"/>
 
@@ -31,15 +32,15 @@
             android:layout_toRightOf="@+id/contact_tmb"
             android:ellipsize="end"
             android:maxLines="1"
-            android:textSize="@dimen/bigger_text_size"
+            android:textSize="@dimen/big_text_size"
             tools:text="John Doe"/>
 
         <TextView
             android:id="@+id/contact_number"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            android:layout_alignLeft="@+id/contact_name"
             android:layout_below="@+id/contact_name"
+            android:layout_alignLeft="@+id/contact_name"
             android:layout_toRightOf="@+id/contact_tmb"
             android:maxLines="1"
             android:textSize="@dimen/bigger_text_size"
diff --git a/app/src/main/res/layout/item_recent_call.xml b/app/src/main/res/layout/item_recent_call.xml
index 61334288..14c60772 100644
--- a/app/src/main/res/layout/item_recent_call.xml
+++ b/app/src/main/res/layout/item_recent_call.xml
@@ -14,16 +14,16 @@
     <android.support.constraint.ConstraintLayout
         android:id="@+id/recent_call_holder"
         android:layout_width="match_parent"
-        android:layout_height="@dimen/contact_item_height"
-        android:gravity="center_vertical"
+        android:layout_height="@dimen/contact_item_with_number_height"
         android:paddingLeft="@dimen/activity_margin"
         android:paddingRight="@dimen/activity_margin">
 
         <TextView
             android:id="@+id/recent_call_name"
             android:layout_width="0dp"
-            android:layout_height="wrap_content"
+            android:layout_height="0dp"
             android:ellipsize="end"
+            android:gravity="center_vertical"
             android:maxLines="1"
             android:paddingRight="@dimen/activity_margin"
             android:textSize="@dimen/big_text_size"
@@ -39,6 +39,7 @@
             android:layout_height="wrap_content"
             android:layout_below="@+id/recent_call_name"
             android:maxLines="1"
+            android:paddingBottom="@dimen/tiny_margin"
             android:textSize="@dimen/bigger_text_size"
             app:layout_constraintBottom_toBottomOf="parent"
             app:layout_constraintEnd_toStartOf="@+id/recent_call_date_time"
diff --git a/app/src/main/res/values/dimens.xml b/app/src/main/res/values/dimens.xml
index e72744fc..eb64d7a4 100644
--- a/app/src/main/res/values/dimens.xml
+++ b/app/src/main/res/values/dimens.xml
@@ -4,6 +4,7 @@
     <dimen name="contact_actions_size">45dp</dimen>
     <dimen name="contact_icons_size">40dp</dimen>
     <dimen name="contact_item_height">52dp</dimen>
+    <dimen name="contact_item_with_number_height">56dp</dimen>
     <dimen name="create_new_contact_height">68dp</dimen>
     <dimen name="dialpad_button_size">60dp</dimen>
     <dimen name="dialpad_text_size">44sp</dimen>