OSDN Git Service

Use drawableStart in dialog list item so we can set padding
authorAlan Viverette <alanv@google.com>
Tue, 12 May 2015 22:25:29 +0000 (15:25 -0700)
committerAlan Viverette <alanv@google.com>
Tue, 12 May 2015 22:25:29 +0000 (15:25 -0700)
And you might think, why does CheckedTextView implement its own drawable
when we have a perfectly good TextView that not only supports drawables
but also supports adjusting the padding?

Good question.

Change-Id: I1ed598846c8a08fffaec9258cf0c3ff86c25353a

core/res/res/layout/select_dialog_multichoice_material.xml
core/res/res/layout/select_dialog_singlechoice_material.xml

index e21df73..36e8e57 100644 (file)
@@ -25,6 +25,6 @@
     android:gravity="center_vertical"
     android:paddingStart="@dimen/select_dialog_padding_start_material"
     android:paddingEnd="?attr/dialogPreferredPadding"
-    android:checkMark="?attr/listChoiceIndicatorMultiple"
-    android:checkMarkGravity="start"
+    android:drawableStart="?attr/listChoiceIndicatorMultiple"
+    android:drawablePadding="20dp"
     android:ellipsize="marquee" />
index 3828317..995272a 100644 (file)
@@ -25,6 +25,6 @@
     android:gravity="center_vertical"
     android:paddingStart="@dimen/select_dialog_padding_start_material"
     android:paddingEnd="?attr/dialogPreferredPadding"
-    android:checkMark="?attr/listChoiceIndicatorSingle"
-    android:checkMarkGravity="start"
+    android:drawableStart="?attr/listChoiceIndicatorSingle"
+    android:drawablePadding="20dp"
     android:ellipsize="marquee" />