From 60b55c919715588965dfc2801e09b15e6c3bea25 Mon Sep 17 00:00:00 2001
From: Sam Bosley <sboz88@gmail.com>
Date: Thu, 14 Jun 2012 20:22:30 -0700
Subject: [PATCH] Polish and layout adjustments to comments list

---
 astrid/res/layout/tag_updates_fragment.xml               | 6 ++++--
 astrid/res/layout/update_adapter_row.xml                 | 7 +++++--
 astrid/res/values/styles.xml                             | 2 +-
 astrid/src/com/todoroo/astrid/adapter/UpdateAdapter.java | 6 +++---
 4 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/astrid/res/layout/tag_updates_fragment.xml b/astrid/res/layout/tag_updates_fragment.xml
index 88f00e38b..8789c3b2d 100644
--- a/astrid/res/layout/tag_updates_fragment.xml
+++ b/astrid/res/layout/tag_updates_fragment.xml
@@ -37,8 +37,10 @@
         android:id="@android:id/list"
         android:layout_width="fill_parent"
         android:layout_height="fill_parent"
-        android:layout_weight="100" />
-
+        android:layout_weight="100" 
+        style="@style/ListView"
+        android:divider="@android:color/transparent"/>
+        
     <ProgressBar
         android:id="@+id/comments_progressBar"
         style="@android:style/Widget.ProgressBar.Horizontal"
diff --git a/astrid/res/layout/update_adapter_row.xml b/astrid/res/layout/update_adapter_row.xml
index eeaa15e05..2ed3a7e34 100644
--- a/astrid/res/layout/update_adapter_row.xml
+++ b/astrid/res/layout/update_adapter_row.xml
@@ -4,6 +4,7 @@
     xmlns:astrid="http://schemas.android.com/apk/res/com.timsu.astrid"
     android:layout_width="fill_parent"
     android:layout_height="wrap_content"
+    android:minHeight="55dip"
     android:orientation="vertical">
 	
 	<RelativeLayout
@@ -43,9 +44,11 @@
             android:layout_width="fill_parent"
             android:layout_height="wrap_content"  
             android:layout_below="@id/title"
-            android:layout_marginLeft="50dip" 
-            android:paddingTop="3dip"
+            android:layout_alignLeft="@id/title"
+            android:layout_marginLeft="54dip" 
+            android:paddingTop="1dip"
             style="@style/TextAppearance.TAd_ItemDueDate"
+            android:textStyle="normal"
             android:gravity="left"
             android:ellipsize="end"
             android:textSize="12sp"
diff --git a/astrid/res/values/styles.xml b/astrid/res/values/styles.xml
index a3ee58178..ec0c54261 100644
--- a/astrid/res/values/styles.xml
+++ b/astrid/res/values/styles.xml
@@ -69,7 +69,7 @@
         <item name="asFilterHeaderColor">#ffffff</item>
         <item name="asFilterHeaderBackground">@drawable/edit_titlebar_white</item>
         <item name="asMembersHeaderBackground">#d0d0d0</item>
-        <item name="asListDividerColor">#cccccc</item>
+        <item name="asListDividerColor">#dddddd</item>
         <item name="asAddButtonImg">@drawable/plus_button_red</item>
         <item name="asMicButtonImg">@drawable/mic_button_red</item>
         <item name="asCommentButtonImg">@drawable/comment_red</item>
diff --git a/astrid/src/com/todoroo/astrid/adapter/UpdateAdapter.java b/astrid/src/com/todoroo/astrid/adapter/UpdateAdapter.java
index 497cdf433..eba04ab68 100644
--- a/astrid/src/com/todoroo/astrid/adapter/UpdateAdapter.java
+++ b/astrid/src/com/todoroo/astrid/adapter/UpdateAdapter.java
@@ -251,9 +251,9 @@ public class UpdateAdapter extends CursorAdapter {
             user = ContextManager.getString(R.string.ENA_no_user);
         }
 
-        String userLink = linkify(user, linkColor);
-        String targetNameLink = linkify(targetName, linkColor);
-        String otherUserLink = linkify(otherUser, linkColor);
+        String userLink = user; //linkify(user, linkColor);
+        String targetNameLink = targetName; //linkify(targetName, linkColor);
+        String otherUserLink = otherUser; //linkify(otherUser, linkColor);
 
         int commentResource = 0;
         if (actionCode.equals(UPDATE_FRIENDS)) {