diff --git a/src/com/android/settings/regionalpreferences/NumberingSystemItemController.java b/src/com/android/settings/regionalpreferences/NumberingSystemItemController.java index e3a8d238c4f..2a99e9925ea 100644 --- a/src/com/android/settings/regionalpreferences/NumberingSystemItemController.java +++ b/src/com/android/settings/regionalpreferences/NumberingSystemItemController.java @@ -153,7 +153,7 @@ public class NumberingSystemItemController extends BasePreferenceController { private void handleLanguageSelect(Preference preference) { String selectedLanguage = preference.getKey(); mMetricsFeatureProvider.action(mContext, - SettingsEnums.ACTION_CHOOSE_LANGUAGE_FOR_NUMBERS_PREFERENCES); + SettingsEnums.ACTION_CHOOSE_LANGUAGE_FOR_NUMBERS_PREFERENCES, selectedLanguage); final Bundle extra = new Bundle(); extra.putString(RegionalPreferencesEntriesFragment.ARG_KEY_REGIONAL_PREFERENCE, ARG_VALUE_NUMBERING_SYSTEM_SELECT); @@ -177,7 +177,8 @@ public class NumberingSystemItemController extends BasePreferenceController { saveNumberingSystemToLocale(Locale.forLanguageTag(mSelectedLanguage), numberingSystem); mMetricsFeatureProvider.action(mContext, - SettingsEnums.ACTION_SET_NUMBERS_PREFERENCES); + SettingsEnums.ACTION_SET_NUMBERS_PREFERENCES, + updatedLocale.getDisplayName() + ": " + numberingSystem); // After updated locale to framework, this fragment will recreate, // so it needs to update the argument of selected language. Bundle bundle = new Bundle(); diff --git a/src/com/android/settings/regionalpreferences/RegionalPreferenceListBasePreferenceController.java b/src/com/android/settings/regionalpreferences/RegionalPreferenceListBasePreferenceController.java index 1e39ffff5e7..432ce0e89e9 100644 --- a/src/com/android/settings/regionalpreferences/RegionalPreferenceListBasePreferenceController.java +++ b/src/com/android/settings/regionalpreferences/RegionalPreferenceListBasePreferenceController.java @@ -59,6 +59,8 @@ public abstract class RegionalPreferenceListBasePreferenceController extends TickButtonPreference pref = new TickButtonPreference(mContext); mPreferenceCategory.addPreference(pref); final String item = unitValues[i]; + final String value = RegionalPreferencesDataUtils.getDefaultUnicodeExtensionData( + mContext, getExtensionTypes()); pref.setTitle(getPreferenceTitle(item)); pref.setKey(item); pref.setOnPreferenceClickListener(clickedPref -> { @@ -66,11 +68,10 @@ public abstract class RegionalPreferenceListBasePreferenceController extends RegionalPreferencesDataUtils.savePreference(mContext, getExtensionTypes(), item.equals(RegionalPreferencesDataUtils.DEFAULT_VALUE) ? null : item); - mMetricsFeatureProvider.action(mContext, getMetricsActionKey()); + mMetricsFeatureProvider.action(mContext, getMetricsActionKey(), + getPreferenceTitle(value) + " > " + getPreferenceTitle(item)); return true; }); - String value = RegionalPreferencesDataUtils.getDefaultUnicodeExtensionData(mContext, - getExtensionTypes()); pref.setSelected(!value.isEmpty() && item.equals(value)); } }