Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 9050c97b authored by Alan Viverette's avatar Alan Viverette
Browse files

Use drawableStart in dialog list item so we can set padding 

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
parent 6dda58e7

core/res/res/layout/select_dialog_multichoice_material.xml

+2 −2
Original line number Diff line number Diff line
@@ -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" />

core/res/res/layout/select_dialog_singlechoice_material.xml

+2 −2
Original line number Diff line number Diff line
@@ -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" />