commit 3827efbe0deb005f45821a3d82de5248adc5ef85
author Sundeep Ghuman <sghuman@google.com> Wed Mar 15 16:40:34 2017 -0700
committer gitbuildkicker <android-build@google.com> Fri Mar 17 18:19:12 2017 -0700
tree 0c793a7d228b8d3ca69de8ba02f4976b3810b7a8
parent 20c14f7a2bfca57bd6025cafe280fa908240260f

Fix list item spacing for preferences.

Bug: b/33970117
Test: Existing tests pass - purely UI margin change. See screenshots in
bug.

Change-Id: I08ce646c9db2401c1462801ab39713fca8f5cc26
(cherry picked from commit 6f98b976871a8a4fb9200a5415aa9d2ac54cb8b6)

res/layout/preference_material_settings.xml

diff --git a/res/layout/preference_material_settings.xml b/res/layout/preference_material_settings.xml
index ae108da..500bc85 100644
--- a/res/layout/preference_material_settings.xml
+++ b/res/layout/preference_material_settings.xml
@@ -20,7 +20,7 @@
     xmlns:android="http://schemas.android.com/apk/res/android"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
-    android:minHeight="?android:attr/listPreferredItemHeight"
+    android:minHeight="?android:attr/listPreferredItemHeightSmall"
     android:gravity="center_vertical"
     android:paddingStart="?android:attr/listPreferredItemPaddingStart"
     android:paddingEnd="?android:attr/listPreferredItemPaddingEnd"
@@ -49,13 +49,14 @@
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:layout_weight="1"
-        android:paddingTop="16dp"
-        android:paddingBottom="16dp">
+        android:paddingTop="12dp"
+        android:paddingBottom="12dp">
 
         <TextView
             android:id="@android:id/title"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
+            android:minHeight="24dp"
             android:singleLine="true"
             android:textAppearance="?android:attr/textAppearanceListItem"
             android:ellipsize="marquee"/>
@@ -64,6 +65,7 @@
             android:id="@android:id/summary"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
+            android:minHeight="24dp"
             android:layout_below="@android:id/title"
             android:layout_alignStart="@android:id/title"
             android:textAppearance="?android:attr/textAppearanceListItemSecondary"