diff options
author | cretin45 <cretin45@gmail.com> | 2016-02-10 13:54:58 -0800 |
---|---|---|
committer | Gerrit Code Review <gerrit@cyanogenmod.org> | 2016-02-11 12:54:49 -0800 |
commit | 9d6893e7ab5737913f2a77db5f4f660033768380 (patch) | |
tree | 010e94c18b6fecf014ee5e7f42fb5c6754f34d94 | |
parent | 523bf5ec6bc06771a019979223b95f35784782eb (diff) | |
download | android_packages_apps_Trebuchet-9d6893e7ab5737913f2a77db5f4f660033768380.tar.gz android_packages_apps_Trebuchet-9d6893e7ab5737913f2a77db5f4f660033768380.tar.bz2 android_packages_apps_Trebuchet-9d6893e7ab5737913f2a77db5f4f660033768380.zip |
Trebuchet: Change icon labels setting to switch
Issue-id: CYNGNOS-1900
Change-Id: Ife68fe84867be631c59cc1de7c44205817e07130
(cherry picked from commit 9c806338cddf3e2a3aff907b821d3abfdb510e72)
-rw-r--r-- | src/com/android/launcher3/list/SettingsPinnedHeaderAdapter.java | 28 |
1 files changed, 17 insertions, 11 deletions
diff --git a/src/com/android/launcher3/list/SettingsPinnedHeaderAdapter.java b/src/com/android/launcher3/list/SettingsPinnedHeaderAdapter.java index 42b52eba0..18d396a10 100644 --- a/src/com/android/launcher3/list/SettingsPinnedHeaderAdapter.java +++ b/src/com/android/launcher3/list/SettingsPinnedHeaderAdapter.java @@ -110,9 +110,8 @@ public class SettingsPinnedHeaderAdapter extends PinnedHeaderListAdapter { current = SettingsProvider.getBoolean(mContext, SettingsProvider.SETTINGS_UI_HOMESCREEN_HIDE_ICON_LABELS, R.bool.preferences_interface_homescreen_hide_icon_labels_default); - state = current ? res.getString(R.string.icon_labels_hide) - : res.getString(R.string.icon_labels_show); - setStateText(stateView, settingSwitch, state); + // Reversed logic here. Boolean is hideLabels, where setting is show labels + setSettingSwitch(stateView, settingSwitch, !current); break; case 2: current = SettingsProvider.getBoolean(mContext, @@ -139,9 +138,8 @@ public class SettingsPinnedHeaderAdapter extends PinnedHeaderListAdapter { current = SettingsProvider.getBoolean(mContext, SettingsProvider.SETTINGS_UI_DRAWER_HIDE_ICON_LABELS, R.bool.preferences_interface_drawer_hide_icon_labels_default); - state = current ? res.getString(R.string.icon_labels_hide) - : res.getString(R.string.icon_labels_show); - setStateText(stateView, settingSwitch, state); + // Reversed logic here. Boolean is hideLabels, where setting is show labels + setSettingSwitch(stateView, settingSwitch, !current); break; case 1: current = SettingsProvider.getBoolean(mContext, @@ -407,7 +405,7 @@ public class SettingsPinnedHeaderAdapter extends PinnedHeaderListAdapter { return !current; } - private void onIconLabelsBooleanChanged(View v, String key, int res) { + private boolean onIconLabelsBooleanChanged(View v, String key, int res) { boolean current = SettingsProvider.getBoolean( mContext, key, res); @@ -415,10 +413,18 @@ public class SettingsPinnedHeaderAdapter extends PinnedHeaderListAdapter { SettingsProvider.putBoolean(mContext, key, !current); SettingsProvider.putBoolean(mContext, SettingsProvider.SETTINGS_CHANGED, true); - String state = current ? mLauncher.getResources().getString( - R.string.icon_labels_show) : mLauncher.getResources().getString( - R.string.icon_labels_hide); - ((TextView) v.findViewById(R.id.item_state)).setText(state); + // Reversed logic here. Boolean is hideLabels, where setting is show labels + ((Switch)v.findViewById(R.id.setting_switch)).setChecked(current); + + Bundle extras = new Bundle(); + extras.putBoolean(LauncherSettings.Settings.EXTRA_VALUE, !current); + + mContext.getContentResolver().call( + LauncherSettings.Settings.CONTENT_URI, + LauncherSettings.Settings.METHOD_SET_BOOLEAN, + key, extras); + + return !current; } private void onDrawerStyleBooleanChanged(View v, String key, int res) { |