diff --git a/src/com/android/settings/accessibility/AccessibilityShortcutPreferenceFragment.java b/src/com/android/settings/accessibility/AccessibilityShortcutPreferenceFragment.java index 34e1fb0557d..5546ba665a3 100644 --- a/src/com/android/settings/accessibility/AccessibilityShortcutPreferenceFragment.java +++ b/src/com/android/settings/accessibility/AccessibilityShortcutPreferenceFragment.java @@ -132,8 +132,8 @@ public abstract class AccessibilityShortcutPreferenceFragment extends DashboardF mShortcutPreference.setPersistent(false); mShortcutPreference.setKey(getShortcutPreferenceKey()); mShortcutPreference.setOnClickCallback(this); + mShortcutPreference.setTitle(getShortcutTitle()); - updateShortcutTitle(mShortcutPreference); getPreferenceScreen().addPreference(mShortcutPreference); mTouchExplorationStateChangeListener = isTouchExplorationEnabled -> { @@ -192,13 +192,11 @@ public abstract class AccessibilityShortcutPreferenceFragment extends DashboardF final Dialog dialog; switch (dialogId) { case DialogEnums.EDIT_SHORTCUT: - final CharSequence dialogTitle = getPrefContext().getString( - R.string.accessibility_shortcut_title, getLabelName()); final int dialogType = WizardManagerHelper.isAnySetupWizard(getIntent()) ? AccessibilityDialogUtils.DialogType.EDIT_SHORTCUT_GENERIC_SUW : AccessibilityDialogUtils.DialogType.EDIT_SHORTCUT_GENERIC; dialog = AccessibilityDialogUtils.showEditShortcutDialog( - getPrefContext(), dialogType, dialogTitle, + getPrefContext(), dialogType, getShortcutTitle(), this::callOnAlertDialogCheckboxClicked); setupEditShortcutDialog(dialog); return dialog; @@ -213,9 +211,8 @@ public abstract class AccessibilityShortcutPreferenceFragment extends DashboardF } } - protected void updateShortcutTitle(ShortcutPreference shortcutPreference) { - final CharSequence title = getString(R.string.accessibility_shortcut_title, getLabelName()); - shortcutPreference.setTitle(title); + protected CharSequence getShortcutTitle() { + return getString(R.string.accessibility_shortcut_title, getLabelName()); } @Override diff --git a/src/com/android/settings/accessibility/ToggleColorInversionPreferenceFragment.java b/src/com/android/settings/accessibility/ToggleColorInversionPreferenceFragment.java index 58837147969..71f47ccbd63 100644 --- a/src/com/android/settings/accessibility/ToggleColorInversionPreferenceFragment.java +++ b/src/com/android/settings/accessibility/ToggleColorInversionPreferenceFragment.java @@ -75,8 +75,8 @@ public class ToggleColorInversionPreferenceFragment extends ToggleFeaturePrefere } @Override - protected void updateShortcutTitle(ShortcutPreference shortcutPreference) { - shortcutPreference.setTitle(R.string.accessibility_display_inversion_shortcut_title); + protected CharSequence getShortcutTitle() { + return getText(R.string.accessibility_display_inversion_shortcut_title); } @Override diff --git a/src/com/android/settings/accessibility/ToggleDaltonizerPreferenceFragment.java b/src/com/android/settings/accessibility/ToggleDaltonizerPreferenceFragment.java index 183da964cfc..ac4408fad89 100644 --- a/src/com/android/settings/accessibility/ToggleDaltonizerPreferenceFragment.java +++ b/src/com/android/settings/accessibility/ToggleDaltonizerPreferenceFragment.java @@ -185,8 +185,8 @@ public final class ToggleDaltonizerPreferenceFragment extends ToggleFeaturePrefe } @Override - protected void updateShortcutTitle(ShortcutPreference shortcutPreference) { - shortcutPreference.setTitle(R.string.accessibility_daltonizer_shortcut_title); + protected CharSequence getShortcutTitle() { + return getText(R.string.accessibility_daltonizer_shortcut_title); } @Override diff --git a/src/com/android/settings/accessibility/ToggleFeaturePreferenceFragment.java b/src/com/android/settings/accessibility/ToggleFeaturePreferenceFragment.java index 248bc12f672..39d20c952f2 100644 --- a/src/com/android/settings/accessibility/ToggleFeaturePreferenceFragment.java +++ b/src/com/android/settings/accessibility/ToggleFeaturePreferenceFragment.java @@ -271,12 +271,10 @@ public abstract class ToggleFeaturePreferenceFragment extends SettingsPreference public Dialog onCreateDialog(int dialogId) { switch (dialogId) { case DialogEnums.EDIT_SHORTCUT: - final CharSequence dialogTitle = getPrefContext().getString( - R.string.accessibility_shortcut_title, mPackageName); final int dialogType = WizardManagerHelper.isAnySetupWizard(getIntent()) ? DialogType.EDIT_SHORTCUT_GENERIC_SUW : DialogType.EDIT_SHORTCUT_GENERIC; mDialog = AccessibilityDialogUtils.showEditShortcutDialog( - getPrefContext(), dialogType, dialogTitle, + getPrefContext(), dialogType, getShortcutTitle(), this::callOnAlertDialogCheckboxClicked); setupEditShortcutDialog(mDialog); return mDialog; @@ -341,9 +339,8 @@ public abstract class ToggleFeaturePreferenceFragment extends SettingsPreference switchPreference.setTitle(title); } - protected void updateShortcutTitle(ShortcutPreference shortcutPreference) { - final CharSequence title = getString(R.string.accessibility_shortcut_title, mPackageName); - shortcutPreference.setTitle(title); + protected CharSequence getShortcutTitle() { + return getString(R.string.accessibility_shortcut_title, mPackageName); } protected void onPreferenceToggled(String preferenceKey, boolean enabled) { @@ -516,8 +513,7 @@ public abstract class ToggleFeaturePreferenceFragment extends SettingsPreference mShortcutPreference.setPersistent(false); mShortcutPreference.setKey(getShortcutPreferenceKey()); mShortcutPreference.setOnClickCallback(this); - - updateShortcutTitle(mShortcutPreference); + mShortcutPreference.setTitle(getShortcutTitle()); final PreferenceCategory generalCategory = findPreference(KEY_GENERAL_CATEGORY); generalCategory.addPreference(mShortcutPreference); diff --git a/src/com/android/settings/accessibility/ToggleReduceBrightColorsPreferenceFragment.java b/src/com/android/settings/accessibility/ToggleReduceBrightColorsPreferenceFragment.java index 93884b766ea..4e990f4d373 100644 --- a/src/com/android/settings/accessibility/ToggleReduceBrightColorsPreferenceFragment.java +++ b/src/com/android/settings/accessibility/ToggleReduceBrightColorsPreferenceFragment.java @@ -168,8 +168,8 @@ public class ToggleReduceBrightColorsPreferenceFragment extends ToggleFeaturePre } @Override - protected void updateShortcutTitle(ShortcutPreference shortcutPreference) { - shortcutPreference.setTitle(R.string.reduce_bright_colors_shortcut_title); + protected CharSequence getShortcutTitle() { + return getText(R.string.reduce_bright_colors_shortcut_title); } @Override diff --git a/src/com/android/settings/accessibility/ToggleScreenMagnificationPreferenceFragment.java b/src/com/android/settings/accessibility/ToggleScreenMagnificationPreferenceFragment.java index bf47658f96b..d9a02c00a8a 100644 --- a/src/com/android/settings/accessibility/ToggleScreenMagnificationPreferenceFragment.java +++ b/src/com/android/settings/accessibility/ToggleScreenMagnificationPreferenceFragment.java @@ -479,8 +479,8 @@ public class ToggleScreenMagnificationPreferenceFragment extends } @Override - protected void updateShortcutTitle(ShortcutPreference shortcutPreference) { - shortcutPreference.setTitle(R.string.accessibility_screen_magnification_shortcut_title); + protected CharSequence getShortcutTitle() { + return getText(R.string.accessibility_screen_magnification_shortcut_title); } @Override diff --git a/src/com/android/settings/gestures/OneHandedSettings.java b/src/com/android/settings/gestures/OneHandedSettings.java index 13ccbd6919b..e1b9b817f95 100644 --- a/src/com/android/settings/gestures/OneHandedSettings.java +++ b/src/com/android/settings/gestures/OneHandedSettings.java @@ -27,7 +27,6 @@ import com.android.internal.accessibility.AccessibilityShortcutController; import com.android.settings.R; import com.android.settings.accessibility.AccessibilityShortcutPreferenceFragment; import com.android.settings.accessibility.AccessibilityUtil.QuickSettingsTooltipType; -import com.android.settings.accessibility.ShortcutPreference; import com.android.settings.search.BaseSearchIndexProvider; import com.android.settingslib.widget.IllustrationPreference; import com.android.settingslib.widget.MainSwitchPreference; @@ -88,8 +87,8 @@ public class OneHandedSettings extends AccessibilityShortcutPreferenceFragment { } @Override - protected void updateShortcutTitle(ShortcutPreference shortcutPreference) { - shortcutPreference.setTitle(R.string.one_handed_mode_shortcut_title); + protected CharSequence getShortcutTitle() { + return getText(R.string.one_handed_mode_shortcut_title); } @Override