From b6acba0e7b329403d481f2de7212ebdc4e6660d2 Mon Sep 17 00:00:00 2001
From: Makoto Onuki <omakoto@google.com>
Date: Thu, 20 Jan 2011 14:42:48 -0800
Subject: [PATCH] Expand Reply/Forward/etc buttons for easier tap

Bug 3372927

Change-Id: I9a6e01c43b4c534b5e6c5c697062313d610c44e6
---
 res/layout-xlarge/message_view_fragment.xml | 28 ++++++++-------------
 res/values-xlarge/styles.xml                |  5 ++++
 2 files changed, 15 insertions(+), 18 deletions(-)

diff --git a/res/layout-xlarge/message_view_fragment.xml b/res/layout-xlarge/message_view_fragment.xml
index 351a64144..bc7b71e53 100644
--- a/res/layout-xlarge/message_view_fragment.xml
+++ b/res/layout-xlarge/message_view_fragment.xml
@@ -96,47 +96,39 @@
                 android:id="@+id/forward_reply_buttons"
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
-                android:layout_marginTop="16dip"
-                android:layout_marginLeft="8dip"
-                android:layout_marginRight="32dip"
+                android:layout_marginTop="8dip"
+                android:layout_marginLeft="0dip"
+                android:layout_marginRight="24dip"
                 android:layout_alignTop="@id/header_background"
                 android:layout_alignRight="@id/header_background"
                 android:orientation="horizontal"
                 >
                 <ImageButton
                     android:id="@+id/reply"
-                    android:layout_width="32dip"
-                    android:layout_height="32dip"
-                    android:layout_marginRight="32dip"
+                    android:layout_marginRight="16dip"
                     android:src="@drawable/ic_reply"
                     android:visibility="gone"
-                    style="@android:style/Widget.Holo.Button.Borderless"
+                    style="@style/message_view_action_buttons"
                     />
                 <ImageButton
                     android:id="@+id/reply_all"
-                    android:layout_width="32dip"
-                    android:layout_height="32dip"
-                    android:layout_marginRight="32dip"
+                    android:layout_marginRight="16dip"
                     android:src="@drawable/ic_reply_all"
                     android:visibility="gone"
-                    style="@android:style/Widget.Holo.Button.Borderless"
+                    style="@style/message_view_action_buttons"
                     />
                 <ImageButton
                     android:id="@+id/forward"
-                    android:layout_width="32dip"
-                    android:layout_height="32dip"
-                    android:layout_marginRight="32dip"
+                    android:layout_marginRight="16dip"
                     android:src="@drawable/ic_forward"
                     android:visibility="gone"
-                    style="@android:style/Widget.Holo.Button.Borderless"
+                    style="@style/message_view_action_buttons"
                     />
                 <ImageButton
                     android:id="@+id/favorite"
-                    android:layout_width="32dip"
-                    android:layout_height="32dip"
                     android:src="@drawable/ic_star_none_holo_light"
                     android:visibility="gone"
-                    style="@android:style/Widget.Holo.Button.Borderless"
+                    style="@style/message_view_action_buttons"
                     />
             </LinearLayout>
 
diff --git a/res/values-xlarge/styles.xml b/res/values-xlarge/styles.xml
index 4b221fed2..6597b5946 100644
--- a/res/values-xlarge/styles.xml
+++ b/res/values-xlarge/styles.xml
@@ -108,4 +108,9 @@
         <item name="android:textColor">@color/text_primary_color</item>
         <item name="android:singleLine">true</item>
     </style>
+    <style name="message_view_action_buttons" parent="android:Widget.Holo.Button.Borderless">
+        <item name="android:layout_width">48dip</item>
+        <item name="android:layout_height">48dip</item>
+        <item name="android:padding">8dip</item>
+    </style>
 </resources>