From 6168534e0a61ca1fe4f8b0db1ddeef4ebfa02fa6 Mon Sep 17 00:00:00 2001
From: Sam Bosley <sboz88@gmail.com>
Date: Fri, 6 Apr 2012 13:12:36 -0700
Subject: [PATCH] Better size for assigned picker rows

---
 .../todoroo/astrid/actfm/EditPeopleControlSet.java | 14 +++++++-------
 astrid/res/layout/assigned_adapter_row.xml         | 10 +++++-----
 astrid/res/layout/control_set_assigned.xml         |  1 -
 astrid/res/values/styles.xml                       |  4 ++++
 4 files changed, 16 insertions(+), 13 deletions(-)

diff --git a/astrid/plugin-src/com/todoroo/astrid/actfm/EditPeopleControlSet.java b/astrid/plugin-src/com/todoroo/astrid/actfm/EditPeopleControlSet.java
index 9d3aa772e..a4a001732 100644
--- a/astrid/plugin-src/com/todoroo/astrid/actfm/EditPeopleControlSet.java
+++ b/astrid/plugin-src/com/todoroo/astrid/actfm/EditPeopleControlSet.java
@@ -569,13 +569,13 @@ public class EditPeopleControlSet extends PopupControlSet {
             }
         });
 
-        assignedCustom.setOnClickListener(new OnClickListener() {
-            @Override
-            public void onClick(View v) {
-                Intent intent = new Intent(Intent.ACTION_PICK, ContactsContract.Contacts.CONTENT_URI);
-                fragment.startActivityForResult(intent, TaskEditFragment.REQUEST_CODE_CONTACT);
-            }
-        });
+//        assignedCustom.setOnClickListener(new OnClickListener() {
+//            @Override
+//            public void onClick(View v) {
+//                Intent intent = new Intent(Intent.ACTION_PICK, ContactsContract.Contacts.CONTENT_URI);
+//                fragment.startActivityForResult(intent, TaskEditFragment.REQUEST_CODE_CONTACT);
+//            }
+//        });
     }
 
     // --- events
diff --git a/astrid/res/layout/assigned_adapter_row.xml b/astrid/res/layout/assigned_adapter_row.xml
index 95dbffd16..a8644ccda 100644
--- a/astrid/res/layout/assigned_adapter_row.xml
+++ b/astrid/res/layout/assigned_adapter_row.xml
@@ -2,12 +2,12 @@
 <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:astrid="http://schemas.android.com/apk/res/com.timsu.astrid"
     android:layout_width="fill_parent"
-    android:layout_height="fill_parent"
+    android:layout_height="wrap_content"
     android:paddingLeft="8dip">
 
     <com.todoroo.astrid.helper.AsyncImageView android:id="@+id/person_image"
-        android:layout_width="40dip"
-        android:layout_height="44dip"
+        android:layout_width="35dip"
+        android:layout_height="35dip"
         android:gravity="center"
         android:layout_gravity="center_vertical"
         android:scaleType="fitCenter"
@@ -15,8 +15,8 @@
 	<CheckedTextView
 		android:id="@android:id/text1"
 		android:layout_width="match_parent"
-		android:layout_height="?android:attr/listPreferredItemHeight"
-		android:textAppearance="?android:attr/textAppearanceLarge"
+		android:layout_height="wrap_content"
+		style="@style/TextAppearance.TEA_assigned"
 		android:textColor="?attr/asTextColor"
 		android:gravity="center_vertical"
 		android:checkMark="?android:attr/listChoiceIndicatorSingle"
diff --git a/astrid/res/layout/control_set_assigned.xml b/astrid/res/layout/control_set_assigned.xml
index 76a460c9d..cb2c73661 100644
--- a/astrid/res/layout/control_set_assigned.xml
+++ b/astrid/res/layout/control_set_assigned.xml
@@ -33,6 +33,5 @@
 	    android:layout_width="fill_parent"
 	    android:layout_height="fill_parent"
 	    android:layout_weight="100"/>
-	<include layout="@layout/control_dialog_ok"/>
     
 </LinearLayout>
diff --git a/astrid/res/values/styles.xml b/astrid/res/values/styles.xml
index 2cdf5d88f..5c99490fa 100644
--- a/astrid/res/values/styles.xml
+++ b/astrid/res/values/styles.xml
@@ -288,6 +288,10 @@
     
     <style name="TextAppearance.GEN_EditLabel.DLG_EditLabel">
     </style>
+
+    <style name="TextAppearance.TEA_assigned">
+        <item name="android:textSize">18sp</item>
+    </style>
    	
    	<style name="Theme.WhenDialog">
    	    <item name="android:windowAnimationStyle">@style/TEA_WhenDialog_Anim</item>