From 2af21e0bfabf04578623ff31e4fa77e955588968 Mon Sep 17 00:00:00 2001
From: Alan Viverette <alanv@google.com>
Date: Wed, 19 Nov 2014 17:17:56 -0800
Subject: [PATCH] Add padding above and below dialog lists to match Material
 spec

There's a little too much padding between lists and buttons, but it's
definitely the lesser of two evils. We can handle it dynamically later
on, but it's tricky since we can't just insert a Space element.

BUG: 18336555
Change-Id: I1a8ff644dd27f7998ce7cf7a4fadba1fd7ed04ff
---
 core/res/res/layout/select_dialog_material.xml | 9 ++++++---
 core/res/res/values/dimens_material.xml        | 3 +++
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/core/res/res/layout/select_dialog_material.xml b/core/res/res/layout/select_dialog_material.xml
index ee040392f2c5b..19ad407d7a966 100644
--- a/core/res/res/layout/select_dialog_material.xml
+++ b/core/res/res/layout/select_dialog_material.xml
@@ -22,11 +22,14 @@
 -->
 <view class="com.android.internal.app.AlertController$RecycleListView"
     xmlns:android="http://schemas.android.com/apk/res/android"
-    android:id="@+android:id/select_dialog_listview"
+    android:id="@id/select_dialog_listview"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
     android:cacheColorHint="@null"
-    android:divider="?android:attr/listDividerAlertDialog"
+    android:divider="?attr/listDividerAlertDialog"
     android:scrollbars="vertical"
     android:overScrollMode="ifContentScrolls"
-    android:textAlignment="viewStart" />
+    android:textAlignment="viewStart"
+    android:paddingTop="@dimen/dialog_list_padding_vertical_material"
+    android:paddingBottom="@dimen/dialog_list_padding_vertical_material"
+    android:clipToPadding="false" />
diff --git a/core/res/res/values/dimens_material.xml b/core/res/res/values/dimens_material.xml
index 4a2119bf4a0e8..7d79ddbd281f0 100644
--- a/core/res/res/values/dimens_material.xml
+++ b/core/res/res/values/dimens_material.xml
@@ -93,4 +93,7 @@
 
     <dimen name="dialog_padding_material">24dp</dimen>
     <dimen name="dialog_padding_top_material">18dp</dimen>
+
+    <!-- Padding above and below selection dialog lists. -->
+    <dimen name="dialog_list_padding_vertical_material">8dp</dimen>
 </resources>