From dca2806a23d202384eb9b37a5026325f8fb08476 Mon Sep 17 00:00:00 2001 From: Fabrice Di Meglio Date: Fri, 21 Feb 2014 17:42:56 -0800 Subject: [PATCH] Implement Holo Light Theme for Settings ... and use Holo (Dark) Theme for the Drawer as requested by UX Also remove the Wi-Fi and Bluetooth switches in the Drawer per following UX specification. Change-Id: I4fc17481255b5db337a887033bc831ded0d2d701 --- res/layout/settings_main.xml | 2 +- res/values/colors.xml | 2 ++ res/values/themes.xml | 3 +-- src/com/android/settings/SettingsActivity.java | 12 +++++++++--- 4 files changed, 13 insertions(+), 6 deletions(-) diff --git a/res/layout/settings_main.xml b/res/layout/settings_main.xml index f34ece82e31..8b98794ab9e 100644 --- a/res/layout/settings_main.xml +++ b/res/layout/settings_main.xml @@ -76,6 +76,6 @@ android:layout_width="300dp" android:layout_height="match_parent" android:layout_gravity="start" - android:background="#ff333333"/> + android:background="@color/background_drawer"/> diff --git a/res/values/colors.xml b/res/values/colors.xml index 40098f5dc75..206dbacb3c4 100644 --- a/res/values/colors.xml +++ b/res/values/colors.xml @@ -15,6 +15,8 @@ --> + #ff777777 + #000 #F00 diff --git a/res/values/themes.xml b/res/values/themes.xml index a87ad33febf..20cb3b47ebd 100644 --- a/res/values/themes.xml +++ b/res/values/themes.xml @@ -59,7 +59,7 @@ @drawable/wifi_signal_light - diff --git a/src/com/android/settings/SettingsActivity.java b/src/com/android/settings/SettingsActivity.java index beb79859114..34c57043cbb 100644 --- a/src/com/android/settings/SettingsActivity.java +++ b/src/com/android/settings/SettingsActivity.java @@ -65,6 +65,7 @@ import android.util.Log; import android.util.Pair; import android.util.TypedValue; import android.util.Xml; +import android.view.ContextThemeWrapper; import android.view.LayoutInflater; import android.view.MenuItem; import android.view.View; @@ -519,7 +520,14 @@ public class SettingsActivity extends Activity DevicePolicyManager dpm = (DevicePolicyManager) getSystemService(Context.DEVICE_POLICY_SERVICE); - mHeaderAdapter= new HeaderAdapter(this, getHeaders(), mAuthenticatorHelper, dpm); + + // As the Settings Theme is now Holo Light, the primary text color is "Black" ... but + // we want the text color of the Drawer items to be "White", so use the inverse Theme (Holo) + // for the Header adapter (and thus making the TextView appearance to have a white color. + Context headersContext = new ContextThemeWrapper(this, + com.android.internal.R.style.Theme_Holo); + + mHeaderAdapter= new HeaderAdapter(headersContext, getHeaders(), mAuthenticatorHelper, dpm); mDevelopmentPreferences = getSharedPreferences(DevelopmentSettings.PREF_FILE, Context.MODE_PRIVATE); @@ -1438,8 +1446,6 @@ public class SettingsActivity extends Activity static int getHeaderType(Header header) { if (header.fragment == null && header.intent == null) { return HEADER_TYPE_CATEGORY; - } else if (header.id == R.id.wifi_settings || header.id == R.id.bluetooth_settings) { - return HEADER_TYPE_SWITCH; } else if (header.id == R.id.security_settings) { return HEADER_TYPE_BUTTON; } else {