Update compose layout to put add photo/add video next to each other

Change-Id: I85723237bc12d74848262ba5dcee62884ec6cd60

1 files changed, 50 insertions, 61 deletions

diff --git a/res/layout-sw600dp/compose.xml b/res/layout-sw600dp/compose.xml
index c09db798c..02318b2ea 100644
--- a/res/layout-sw600dp/compose.xml
+++ b/res/layout-sw600dp/compose.xml
@@ -38,16 +38,13 @@
             android:background="@android:color/white"
             android:layout_gravity="center_horizontal"
             android:paddingLeft="@dimen/compose_area_left_padding"
+            android:paddingRight="@dimen/compose_area_right_padding"
             android:animateLayoutChanges="true">
             <TableRow
                 android:layout_width="match_parent"
                 android:layout_height="wrap_content">
                 <FrameLayout
-                    android:layout_width="0dip"
-                    android:layout_weight="1"
-                    android:layout_height="wrap_content"
-                    android:layout_column="1"
-                    android:layout_marginRight="@dimen/compose_area_right_padding">
+                    style="@style/FillRowStyle">
                     <include layout="@layout/compose_from"/>
                 </FrameLayout>
             </TableRow>
@@ -56,15 +53,18 @@
                 android:layout_height="wrap_content">
                 <LinearLayout
                     android:id="@+id/to_content"
-                    style="@style/RecipientComposeFieldLayout"
                     android:layout_column="1"
                     android:layout_width="0dip"
-                    android:layout_weight="1">
-                        <LinearLayout android:layout_width="match_parent"
+                    android:layout_weight="1"
+                    style="@style/RecipientComposeFieldLayout"
+                    android:paddingRight="8dp"
+                    android:orientation="horizontal">
+                        <LinearLayout android:layout_width="0dip"
                             android:layout_height="wrap_content"
-                            android:addStatesFromChildren="true"
+                            android:layout_weight="1"
                             android:gravity="center_vertical"
-                            android:layout_gravity="center_vertical">
+                            android:layout_gravity="center_vertical"
+                            android:addStatesFromChildren="true">
 
                             <TextView style="@style/RecipientComposeHeading"
                                 android:text="@string/to"
@@ -78,29 +78,23 @@
 
                         </LinearLayout>
 
-                        <View style="@style/RecipientComposeFieldSpacer"/>
+                        <Button android:id="@+id/add_cc_bcc"
+                            android:text="@string/add_cc_label"
+                            style="@style/ComposeButton"
+                            android:layout_width="wrap_content"
+                            android:layout_column="2"
+                            android:layout_gravity="bottom"
+                            android:gravity="bottom"
+                            android:layout_marginTop="16dip"
+                            android:paddingLeft="8dip"/>
                 </LinearLayout>
-                <Button android:id="@+id/add_cc_bcc"
-                    android:text="@string/add_cc_label"
-                    style="@style/ComposeButton"
-                    android:layout_width="wrap_content"
-                    android:minWidth="@dimen/compose_button_width"
-                    android:layout_column="2"
-                    android:layout_gravity="center_vertical|left"
-                    android:gravity="left"
-                    android:layout_marginTop="16dip"
-                    android:paddingLeft="8dip"/>
             </TableRow>
             <TableRow
                 android:layout_width="match_parent"
-                android:layout_height="wrap_content"
-                android:layout_marginRight="@dimen/compose_button_width">
+                android:layout_height="wrap_content">
                 <com.android.mail.compose.CcBccView
-                    android:layout_height="wrap_content"
                     android:id="@+id/cc_bcc_wrapper"
-                    android:layout_column="1"
-                    android:layout_width="0dip"
-                    android:layout_weight="1"/>
+                    style="@style/FillRowStyle"/>
             </TableRow>
             <TableRow
                 android:layout_width="match_parent"
@@ -115,53 +109,53 @@
                     <com.android.mail.compose.EnterSubject android:id="@+id/subject"
                         style="@style/ComposeSubjectView" />
                 </RelativeLayout>
-                <ImageView android:id="@+id/add_photo_attachment"
-                        android:text="@string/add_file_attachment"
-                        android:layout_width="@dimen/compose_button_width"
-                        android:layout_height="wrap_content"
-                        android:src="@drawable/ic_attach_image_holo_light"
-                        android:clickable="true"
-                        android:layout_gravity="center_vertical|left"
-                        android:gravity="left"
-                        android:layout_column="2" />
+                <RelativeLayout android:layout_width="wrap_content"
+                    android:layout_height="wrap_content"
+                    android:layout_column="2">
+                    <RelativeLayout android:id="@+id/add_photo_attachment"
+                        style="@style/AttachmentButtonStyle">
+                        <ImageView
+                            android:text="@string/add_file_attachment"
+                            android:src="@drawable/ic_attach_image_holo_light"
+                            android:clickable="false"
+                            android:layout_width="wrap_content"
+                            android:layout_height="wrap_content"/>
+                    </RelativeLayout>
+                    <RelativeLayout android:id="@+id/add_video_attachment"
+                        android:layout_toRightOf="@id/add_photo_attachment"
+                        style="@style/AttachmentButtonStyle">
+                        <ImageView
+                            android:text="@string/add_file_attachment"
+                            android:layout_width="wrap_content"
+                            android:layout_height="wrap_content"
+                            android:src="@drawable/ic_attach_video_holo_light"
+                            android:clickable="false"/>
+                    </RelativeLayout>
+                </RelativeLayout>
             </TableRow>
             <TableRow
                 android:layout_width="match_parent"
                 android:layout_height="wrap_content">
                 <!-- Compose Area -->
-                <FrameLayout android:layout_width="0dip"
-                    android:layout_weight="1"
-                    android:layout_height="wrap_content"
+                <FrameLayout
                     android:id="@+id/compose_wrapper"
-                    android:layout_column="1">
+                    style="@style/FillRowStyle">
                     <include layout="@layout/compose_body"/>
                 </FrameLayout>
-                <ImageView android:id="@+id/add_video_attachment"
-                    android:text="@string/add_file_attachment"
-                    android:layout_width="@dimen/compose_button_width"
-                    android:src="@drawable/ic_attach_video_holo_light"
-                    android:clickable="true"
-                    android:layout_gravity="center_vertical|left"
-                    android:gravity="left"
-                    android:layout_column="2"/>
             </TableRow>
             <TableRow
                 android:layout_width="match_parent"
                 android:layout_height="wrap_content">
                 <!--  Attachments -->
                 <com.android.mail.compose.AttachmentsView android:id="@+id/attachments"
-                    android:layout_height="wrap_content"
                     android:orientation="vertical"
                     android:layout_marginTop="8dip"
-                    android:layout_marginRight="@dimen/compose_area_right_padding"
                     android:animateLayoutChanges="true"
                     android:paddingTop="2dip"
                     android:paddingBottom="0dip"
                     android:paddingLeft="0dip"
                     android:visibility="gone"
-                    android:layout_column="1"
-                    android:layout_weight="1"
-                    android:layout_width="0dip" >
+                    style="@style/FillRowStyle">
                     <include layout="@layout/compose_attachments" />
                 </com.android.mail.compose.AttachmentsView>
             </TableRow>
@@ -172,20 +166,15 @@
                 <com.android.mail.compose.QuotedTextView android:id="@+id/quoted_text_view"
                     android:layout_height="wrap_content"
                     android:visibility="gone"
-                    android:layout_marginRight="@dimen/compose_area_right_padding"
-                    android:layout_column="1"
-                    android:layout_weight="1"
-                    android:layout_width="0dip"/>
+                    style="@style/FillRowStyle"/>
             </TableRow>
             <TableRow
                 android:layout_width="match_parent"
                 android:layout_height="wrap_content">
                 <FrameLayout
                     android:id="@+id/composearea_tap_trap_bottom"
-                    android:layout_weight="1"
-                    android:layout_height="wrap_content"
-                    android:layout_width="0dip"
-                    android:clickable="true" android:layout_column="1"/>
+                    style="@style/FillRowStyle"
+                    android:clickable="true"/>
             </TableRow>
 
         </TableLayout>