From 1c118b5f2a9d88b51c0c6a66443ff439ca377cab Mon Sep 17 00:00:00 2001 From: Fan Zhang Date: Fri, 9 Dec 2016 13:04:24 -0800 Subject: [PATCH] Always reserve space for preference icons. This aligns text on preferences on the same screen when some of them don't have icons. - Use a custom layout for preference EditPreference switch preference VolumeSeekPreference DialogPreferencei Restricted*Preference DropdownPreference - Moved related style defs into a new file styles_preference.xml, because current styles file is too large to read. - Switch to for a bunch of regular preferences. - Also removed some dead style. Bug: 33430928 Test: runtest --path packages/apps/Settings/tests/app/src/com/android/settings/dashboard/PreferenceThemeTest.java Change-Id: Ic9b75d3f133f2a83cd999297b8596253d4b10161 --- .../preference_category_material_settings.xml | 51 +++++++++++ .../preference_dropdown_material_settings.xml | 89 +++++++++++++++++++ res/layout/preference_material_settings.xml | 26 +++--- res/layout/preference_volume_slider.xml | 63 +++++++------ res/values/dimens.xml | 1 - res/values/styles.xml | 32 ------- res/values/styles_preference.xml | 58 ++++++++++++ res/values/themes.xml | 14 ++- res/xml/about_legal.xml | 29 +++--- res/xml/accessibility_settings.xml | 16 ++-- res/xml/advanced_apps.xml | 5 +- res/xml/app_default_settings.xml | 3 +- res/xml/date_time_prefs.xml | 2 +- res/xml/development_prefs.xml | 16 ++-- res/xml/device_info_settings.xml | 36 ++++---- res/xml/ia_display_settings.xml | 6 +- res/xml/security_settings_misc.xml | 2 +- res/xml/sound_settings.xml | 10 +-- .../android/settings/DevelopmentSettings.java | 7 +- .../accessibility/AccessibilitySettings.java | 36 ++++---- .../display/ScreenZoomPreference.java | 17 ++-- .../dashboard/FirstIdViewMatcher.java | 51 +++++++++++ .../dashboard/PreferenceThemeTest.java | 68 ++++++++++++++ 23 files changed, 465 insertions(+), 173 deletions(-) create mode 100644 res/layout/preference_category_material_settings.xml create mode 100644 res/layout/preference_dropdown_material_settings.xml create mode 100644 res/values/styles_preference.xml create mode 100644 tests/app/src/com/android/settings/dashboard/FirstIdViewMatcher.java create mode 100644 tests/app/src/com/android/settings/dashboard/PreferenceThemeTest.java diff --git a/res/layout/preference_category_material_settings.xml b/res/layout/preference_category_material_settings.xml new file mode 100644 index 00000000000..13cefdbf05b --- /dev/null +++ b/res/layout/preference_category_material_settings.xml @@ -0,0 +1,51 @@ + + + + + + + + + + + + + \ No newline at end of file diff --git a/res/layout/preference_dropdown_material_settings.xml b/res/layout/preference_dropdown_material_settings.xml new file mode 100644 index 00000000000..ee4051da87e --- /dev/null +++ b/res/layout/preference_dropdown_material_settings.xml @@ -0,0 +1,89 @@ + + + + + + + + + + + + + + + + + + + + + + + + + \ No newline at end of file diff --git a/res/layout/preference_material_settings.xml b/res/layout/preference_material_settings.xml index b8a00f2783d..ae108da5c7a 100644 --- a/res/layout/preference_material_settings.xml +++ b/res/layout/preference_material_settings.xml @@ -16,18 +16,19 @@ - + android:maxHeight="48dp"/> - + android:ellipsize="marquee"/> - + android:maxLines="10"/> - + android:orientation="vertical"/> diff --git a/res/layout/preference_volume_slider.xml b/res/layout/preference_volume_slider.xml index c5c7818d8ed..3eb0c97257f 100644 --- a/res/layout/preference_volume_slider.xml +++ b/res/layout/preference_volume_slider.xml @@ -14,14 +14,32 @@ limitations under the License. --> - + android:clickable="false" + android:orientation="horizontal"> + + + + - + + android:fadingEdge="horizontal"/> - + android:orientation="vertical"/> - - - + android:layout_height="wrap_content"/> - + android:textColor="?android:attr/textColorSecondary"/> - - + diff --git a/res/values/dimens.xml b/res/values/dimens.xml index 40f3f71ae52..2931a989a92 100755 --- a/res/values/dimens.xml +++ b/res/values/dimens.xml @@ -51,7 +51,6 @@ 3dip 6dip 7dip - 1dip 28dp diff --git a/res/values/styles.xml b/res/values/styles.xml index a18fde3f5e2..f4b39d42c22 100644 --- a/res/values/styles.xml +++ b/res/values/styles.xml @@ -93,22 +93,6 @@ @null - - - - - - - - - - - - @@ -324,10 +296,6 @@ 33554432 - - diff --git a/res/values/styles_preference.xml b/res/values/styles_preference.xml new file mode 100644 index 00000000000..a504166d469 --- /dev/null +++ b/res/values/styles_preference.xml @@ -0,0 +1,58 @@ + + + + + + + + + + + + + + + + + + + + + + \ No newline at end of file diff --git a/res/values/themes.xml b/res/values/themes.xml index 6b765f7e9cf..edc1dbf616a 100644 --- a/res/values/themes.xml +++ b/res/values/themes.xml @@ -88,11 +88,16 @@