Fix "manage-conference" button layout.
Daisuke Miyakawa [Mon, 12 Mar 2012 20:56:38 +0000 (13:56 -0700)]
1. Stop using InCallButton and have separate style for extra buttons

InCallButton and other extra buttons have different design now.

2. Stop specifying ImageButton's background

LinearLayout outside the ImageButton already specifies that, and
specifying the same background twice causes strange effect now.

3. set horizontal padding toward LinearLayout for each button,
   not FrameLayou containing them.

This allows wider touch effect reaching to the device edges.

Bug: 6154379
Change-Id: Ibcd845ecbceb730bffcef73cae70f3c941ae93c0

res/layout/extra_button_row.xml
res/layout/incall_touch_ui.xml
res/values/dimens.xml
res/values/styles.xml

index fb6da6a..1144661 100644 (file)
         android:orientation="horizontal"
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
+        android:paddingLeft="@dimen/button_cluster_side_padding"
+        android:paddingRight="@dimen/button_cluster_side_padding"
         android:background="?android:attr/selectableItemBackground">
         <!-- The entire LinearLayout here is clickable, so we don't
              care about clicks on the ImageButton itself. -->
         <ImageButton android:id="@+id/manageConferenceButtonImage"
             android:clickable="false"
-            style="@style/InCallButton"
+            style="@style/InCallExtraRowButton"
             android:src="@drawable/ic_groups_holo_dark"
             android:contentDescription="@string/onscreenManageConferenceText" />
         <TextView android:id="@+id/manageConferenceButtonLabel"
         android:orientation="horizontal"
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
+        android:paddingLeft="@dimen/button_cluster_side_padding"
+        android:paddingRight="@dimen/button_cluster_side_padding"
         android:background="?android:attr/selectableItemBackground">
         <!-- The entire LinearLayout here is clickable, so we don't
              care about clicks on the ImageButton itself. -->
         <ImageButton android:id="@+id/cdmaMergeButtonImage"
             android:clickable="false"
-            style="@style/InCallButton"
+            style="@style/InCallExtraRowButton"
             android:src="@drawable/ic_merge_holo_dark"
             android:contentDescription="@string/onscreenMergeCallsText" />
         <TextView android:id="@+id/cdmaMergeButtonLabel"
index 2fd3de8..960287e 100644 (file)
@@ -65,9 +65,7 @@
             <ViewStub android:id="@+id/extraButtonRow"
                       android:layout="@layout/extra_button_row"
                       android:layout_width="match_parent"
-                      android:layout_height="wrap_content"
-                      android:paddingLeft="@dimen/button_cluster_side_padding"
-                      android:paddingRight="@dimen/button_cluster_side_padding" />
+                      android:layout_height="wrap_content" />
 
             <!-- Row 2: The "End call" button. -->
             <ImageButton android:id="@+id/endButton"
index f324573..9ed195c 100644 (file)
     <!-- Height of the in-call "End" button. -->
     <dimen name="in_call_end_button_height">74dp</dimen>
 
+    <!-- Width of buttons in the extra button row. -->
+    <dimen name="extra_row_button_width">56dp</dimen>
+
+    <!-- Height of buttons in the extra button row. -->
+    <dimen name="extra_row_button_height">@dimen/in_call_button_height</dimen>
+
     <!-- Padding at the left and right edges of the incall_touch_ui button
          cluster.  This padding is necessary because we can't allow the
          buttons to be very close to the edges of the screen, due to the
index ed29b0a..8d73d0d 100644 (file)
         <item name="android:background">?android:attr/selectableItemBackground</item>
     </style>
 
+    <!-- Buttons in extra button row. -->
+    <style name="InCallExtraRowButton">
+        <item name="android:layout_width">@dimen/extra_row_button_width</item>
+        <item name="android:layout_height">@dimen/extra_row_button_height</item>
+        <item name="android:background">@null</item>
+    </style>
+
     <!-- Text label drawn alongside buttons in the "extra button row" of
          the in-call onscreen touch UI. -->
     <style name="InCallExtraRowButtonLabel">