Sets the minimum height on the ActionsViewContainer.
Since the frame layout containing the actions had a minimum height but
not the ActionsViewContainer itself, it was possible to have extra space
around it which would not be highlighted correctly.
Moving the minimum height to the ActionsViewContainer itself means that
there will never be extra padding from the FrameLayout.
Also remove the declared top and bottom padding since these are always
set programmatically anyway.
Bug: 5341956
Change-Id: I0345e1bf6f27cb92c499adf0b1e9ce1363155058
diff --git a/res/layout/contact_detail_list_item.xml b/res/layout/contact_detail_list_item.xml
index ece0064..a410dc9 100644
--- a/res/layout/contact_detail_list_item.xml
+++ b/res/layout/contact_detail_list_item.xml
@@ -22,11 +22,7 @@
xmlns:android="http://schemas.android.com/apk/res/android"
android:layout_width="match_parent"
android:layout_height="wrap_content"
- android:gravity="center_vertical"
- android:paddingLeft="16dip"
- android:paddingTop="8dip"
- android:paddingBottom="8dip"
- android:minHeight="@dimen/detail_min_line_item_height">
+ android:paddingLeft="16dip">
<com.android.contacts.detail.ActionsViewContainer
android:id="@+id/actions_view_container"
@@ -35,7 +31,8 @@
android:orientation="horizontal"
android:focusable="true"
android:background="?android:attr/selectableItemBackground"
- android:nextFocusRight="@+id/secondary_action_view_container">
+ android:nextFocusRight="@+id/secondary_action_view_container"
+ android:minHeight="@dimen/detail_min_line_item_height">
<!-- Note: padding might be controlled programatically -->
<LinearLayout