From d7970b3635cecb51512b015fc9c136e337ffc3b0 Mon Sep 17 00:00:00 2001
From: Mark Wei <markwei@google.com>
Date: Tue, 2 Apr 2013 14:50:52 -0700
Subject: [PATCH] Attachments in Conversation List: UI changes to display
 attachment previews in the conversation list.

Depends on:
PhotoManager: https://googleplex-android-review.googlesource.com/#/c/285814/

Bug: 7481049
Change-Id: Ia9f93c4ce75a4c9be0e9b738d0347292e4b2faca
---
 res/layout/conversation_item_view_normal.xml |   8 ++
 res/layout/conversation_item_view_wide.xml   | 139 ++++++++++---------
 2 files changed, 84 insertions(+), 63 deletions(-)

diff --git a/res/layout/conversation_item_view_normal.xml b/res/layout/conversation_item_view_normal.xml
index 276ab166f..bbe796c13 100644
--- a/res/layout/conversation_item_view_normal.xml
+++ b/res/layout/conversation_item_view_normal.xml
@@ -99,4 +99,12 @@
             style="@style/StarStyle"
             android:src="@drawable/btn_star_off_normal_email_holo_light" />
     </LinearLayout>
+    <ImageView
+        android:id="@+id/attachment_previews"
+        android:layout_width="match_parent"
+        android:layout_height="0dp"
+        android:layout_marginTop="@dimen/attachment_preview_margin_top"
+        android:layout_marginLeft="@dimen/attachment_preview_margin_side"
+        android:layout_marginRight="@dimen/attachment_preview_margin_side"
+        android:visibility="gone" />
 </LinearLayout>
diff --git a/res/layout/conversation_item_view_wide.xml b/res/layout/conversation_item_view_wide.xml
index 3cae7f6c6..5a656d2a8 100644
--- a/res/layout/conversation_item_view_wide.xml
+++ b/res/layout/conversation_item_view_wide.xml
@@ -26,75 +26,88 @@
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
     android:minHeight="64sp"
-    android:orientation="horizontal">
-    <ImageView
-        android:id="@+id/checkmark"
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:layout_marginLeft="16dip"
-        android:layout_marginRight="16dip"
-        android:layout_gravity="center_vertical"
-        android:src="@drawable/btn_check_on_normal_holo_light" />
-    <TextView
-        android:id="@+id/senders"
-        android:layout_width="224dip"
-        android:layout_height="wrap_content"
-        android:text="@string/long_string"
-        android:textSize="@dimen/wide_senders_font_size"
-        android:layout_gravity="center_vertical"
-        android:maxLines="2"
-        android:layout_marginTop="@dimen/wide_senders_margin_top"  />
-    <ImageView
-        android:id="@+id/reply_state"
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:layout_marginLeft="16dip"
-        android:layout_marginRight="16dip"
-        android:layout_gravity="center_vertical"
-        android:src="@drawable/ic_badge_reply_holo_light" />
-    <TextView
-        android:id="@+id/subject"
-        android:layout_width="0dip"
-        android:layout_weight="0.7"
-        android:layout_height="wrap_content"
-        android:layout_gravity="center_vertical"
-        android:text="@string/long_string"
-        android:lines="2"
-        android:textColor="@color/subject_text_color_unread"
-        android:textSize="@dimen/wide_subject_font_size"
-        android:layout_marginTop="2sp"
-        android:layout_marginRight="@dimen/wide_subject_margin_right"/>
+    android:orientation="vertical" >
     <LinearLayout
-        android:layout_width="wrap_content"
+        android:layout_width="match_parent"
         android:layout_height="match_parent"
-        android:orientation="vertical">
-        <LinearLayout
+        android:orientation="horizontal">
+        <ImageView
+            android:id="@+id/checkmark"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:orientation="horizontal"
-            android:layout_gravity="center_vertical">
-            <ImageView
-                android:id="@+id/paperclip"
+            android:layout_marginLeft="16dip"
+            android:layout_marginRight="16dip"
+            android:layout_gravity="center_vertical"
+            android:src="@drawable/btn_check_on_normal_holo_light" />
+        <TextView
+            android:id="@+id/senders"
+            android:layout_width="224dip"
+            android:layout_height="wrap_content"
+            android:text="@string/long_string"
+            android:textSize="@dimen/wide_senders_font_size"
+            android:layout_gravity="center_vertical"
+            android:maxLines="2"
+            android:layout_marginTop="@dimen/wide_senders_margin_top"  />
+        <ImageView
+            android:id="@+id/reply_state"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_marginLeft="16dip"
+            android:layout_marginRight="16dip"
+            android:layout_gravity="center_vertical"
+            android:src="@drawable/ic_badge_reply_holo_light" />
+        <TextView
+            android:id="@+id/subject"
+            android:layout_width="0dip"
+            android:layout_weight="0.7"
+            android:layout_height="wrap_content"
+            android:layout_gravity="center_vertical"
+            android:text="@string/long_string"
+            android:lines="2"
+            android:textColor="@color/subject_text_color_unread"
+            android:textSize="@dimen/wide_subject_font_size"
+            android:layout_marginTop="2sp"
+            android:layout_marginRight="@dimen/wide_subject_margin_right"/>
+        <LinearLayout
+            android:layout_width="wrap_content"
+            android:layout_height="match_parent"
+            android:orientation="vertical">
+            <LinearLayout
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
-                android:src="@drawable/ic_attachment_holo_light"
-                android:layout_gravity="center_vertical"
-                android:layout_marginTop="@dimen/wide_attachment_margin_top"/>
-            <TextView
-                android:id="@+id/date"
-                android:layout_width="wrap_content"
-                android:layout_height="wrap_content"
-                android:layout_gravity="center_vertical"
-                android:text="@string/date"
-                android:layout_marginTop="@dimen/wide_date_margin_top" />
+                android:orientation="horizontal"
+                android:layout_gravity="center_vertical">
+                <ImageView
+                    android:id="@+id/paperclip"
+                    android:layout_width="wrap_content"
+                    android:layout_height="wrap_content"
+                    android:src="@drawable/ic_attachment_holo_light"
+                    android:layout_gravity="center_vertical"
+                    android:layout_marginTop="@dimen/wide_attachment_margin_top"/>
+                <TextView
+                    android:id="@+id/date"
+                    android:layout_width="wrap_content"
+                    android:layout_height="wrap_content"
+                    android:layout_gravity="center_vertical"
+                    android:text="@string/date"
+                    android:layout_marginTop="@dimen/wide_date_margin_top" />
+            </LinearLayout>
         </LinearLayout>
+        <ImageView
+            android:id="@+id/star"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_marginLeft="16dip"
+            android:layout_marginRight="16dip"
+            android:layout_gravity="center_vertical"
+            android:src="@drawable/btn_star_off_normal_email_holo_light" />
     </LinearLayout>
     <ImageView
-        android:id="@+id/star"
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:layout_marginLeft="16dip"
-        android:layout_marginRight="16dip"
-        android:layout_gravity="center_vertical"
-        android:src="@drawable/btn_star_off_normal_email_holo_light" />
-</LinearLayout>
+        android:id="@+id/attachment_previews"
+        android:layout_width="match_parent"
+        android:layout_height="0dp"
+        android:layout_marginTop="@dimen/attachment_preview_margin_top"
+        android:layout_marginLeft="@dimen/attachment_preview_margin_side"
+        android:layout_marginRight="@dimen/attachment_preview_margin_side"
+        android:visibility="gone" />
+</LinearLayout>
\ No newline at end of file