commit 53fa6ef138f831a5a510d759a752d5101a256dce
author Katherine Kuan <katherinekuan@google.com> Fri Jul 22 13:35:22 2011 -0700
committer Katherine Kuan <katherinekuan@google.com> Fri Jul 22 14:28:09 2011 -0700
tree c0cf7cf2a7a78b06e1706e00fb4d0d5b72ddcc19
parent 402d5c9ccd5d07aca67618de76ddf24d1a9a0e46

Remove account name and icon in group list item

- Redundant info that is already in the list header
- TODO: Still a bunch of work left on how to display
group members and photos

Bug: 5039532
Change-Id: I615144499052028b0f479bc02594b579ba689d32

res/layout/group_browse_list_item.xml

diff --git a/res/layout/group_browse_list_item.xml b/res/layout/group_browse_list_item.xml
index d94d444..b829704 100644
--- a/res/layout/group_browse_list_item.xml
+++ b/res/layout/group_browse_list_item.xml
@@ -22,42 +22,13 @@
     android:layout_height="wrap_content"
     style="@style/GroupBrowseListItem">
 
-    <LinearLayout
-        android:orientation="vertical"
-        android:layout_width="0dip"
-        android:layout_height="match_parent"
-        android:layout_weight="1"
-        android:padding="5dip">
-
-        <TextView
-            android:id="@+id/label"
-            android:layout_height="wrap_content"
-            android:layout_width="wrap_content"
-            android:gravity="center_vertical"
-            android:textAppearance="?android:attr/textAppearanceLarge"
-            android:ellipsize="end"
-            android:singleLine="true" />
-
-        <TextView
-            android:id="@+id/account"
-            android:layout_height="wrap_content"
-            android:layout_width="wrap_content"
-            android:gravity="center_vertical"
-            android:textAppearance="?android:attr/textAppearanceSmall"
-            android:textColor="?android:attr/textColorSecondary"
-            android:ellipsize="end"
-            android:singleLine="true"
-            android:textStyle="italic" />
-
-    </LinearLayout>
-
-    <ImageView
-        android:id="@+id/icon"
-        android:scaleType="center"
+    <TextView
+        android:id="@+id/label"
+        android:layout_height="wrap_content"
         android:layout_width="wrap_content"
-        android:layout_height="match_parent"
-        android:layout_marginRight="20dip"
-        android:layout_gravity="center_vertical"
-        android:src="@drawable/ic_menu_display_all_holo_light" />
+        android:padding="10dip"
+        android:textAppearance="?android:attr/textAppearanceLarge"
+        android:ellipsize="end"
+        android:singleLine="true" />
 
 </view>

src/com/android/contacts/group/GroupBrowseListAdapter.java

diff --git a/src/com/android/contacts/group/GroupBrowseListAdapter.java b/src/com/android/contacts/group/GroupBrowseListAdapter.java
index cc15213..2dd194d 100644
--- a/src/com/android/contacts/group/GroupBrowseListAdapter.java
+++ b/src/com/android/contacts/group/GroupBrowseListAdapter.java
@@ -234,7 +234,6 @@
     public static class GroupListItem extends LinearLayout {
 
         private TextView mLabel;
-        private TextView mAccount;
         private Uri mUri;
 
         public GroupListItem(Context context, AttributeSet attrs, int defStyle) {
@@ -253,12 +252,10 @@
         protected void onFinishInflate() {
             super.onFinishInflate();
             mLabel = (TextView) findViewById(R.id.label);
-            mAccount = (TextView) findViewById(R.id.account);
         }
 
         public void loadFromGroup(GroupMetaData group) {
             mLabel.setText(group.getTitle());
-            mAccount.setText(group.getAccountName());
             mUri = getGroupUriFromId(group.getGroupId());
         }