Revert "[Sim UI enhancement] remove the "Tap to show info""

This reverts commit c6eba747db.

Change-Id: Ib8403ca842a4655d1526b27f1965feeee336c687
This commit is contained in:
althafvly
2024-06-27 21:19:52 +05:30
committed by Michael Bestas
parent 119bee2b19
commit 6b959b35d8
4 changed files with 39 additions and 13 deletions

View File

@@ -51,7 +51,7 @@
settings:controller="com.android.settings.deviceinfo.BrandedAccountPreferenceController"/> settings:controller="com.android.settings.deviceinfo.BrandedAccountPreferenceController"/>
<!-- Phone number --> <!-- Phone number -->
<Preference <com.android.settings.deviceinfo.PhoneNumberSummaryPreference
android:key="phone_number" android:key="phone_number"
android:order="3" android:order="3"
android:title="@string/status_number" android:title="@string/status_number"
@@ -119,6 +119,7 @@
android:key="eid_info" android:key="eid_info"
android:order="31" android:order="31"
android:title="@string/status_eid" android:title="@string/status_eid"
android:summary="@string/device_info_protected_single_press"
android:positiveButtonText="@string/dlg_ok" android:positiveButtonText="@string/dlg_ok"
android:dialogLayout="@layout/dialog_eid_status" android:dialogLayout="@layout/dialog_eid_status"
settings:isPreferenceVisible="@bool/config_show_sim_info" settings:isPreferenceVisible="@bool/config_show_sim_info"
@@ -126,7 +127,7 @@
settings:controller="com.android.settings.deviceinfo.simstatus.SimEidPreferenceController"/> settings:controller="com.android.settings.deviceinfo.simstatus.SimEidPreferenceController"/>
<!-- IMEI --> <!-- IMEI -->
<Preference <com.android.settings.deviceinfo.PhoneNumberSummaryPreference
android:key="imei_info" android:key="imei_info"
android:order="32" android:order="32"
android:title="@string/status_imei" android:title="@string/status_imei"

View File

@@ -62,6 +62,27 @@ public class PhoneNumberPreferenceController extends BasePreferenceController {
return AVAILABLE; return AVAILABLE;
} }
@Override
public CharSequence getSummary() {
return mContext.getString(R.string.device_info_protected_single_press);
}
@Override
public boolean handlePreferenceTreeClick(Preference preference) {
String prefKey = preference.getKey();
if (prefKey.startsWith(KEY_PHONE_NUMBER)) {
int simSlotNumber = 0;
if (!TextUtils.equals(prefKey, KEY_PHONE_NUMBER)) {
// Get multisim slot number from preference key.
// Multisim preference key is KEY_PHONE_NUMBER + simSlotNumber
simSlotNumber = Integer.parseInt(
prefKey.replaceAll("[^0-9]", ""));
}
setPhoneNumber(simSlotNumber);
}
return super.handlePreferenceTreeClick(preference);
}
@Override @Override
public void displayPreference(PreferenceScreen screen) { public void displayPreference(PreferenceScreen screen) {
super.displayPreference(screen); super.displayPreference(screen);
@@ -78,7 +99,7 @@ public class PhoneNumberPreferenceController extends BasePreferenceController {
for (int simSlotNumber = 1; simSlotNumber < mTelephonyManager.getPhoneCount(); for (int simSlotNumber = 1; simSlotNumber < mTelephonyManager.getPhoneCount();
simSlotNumber++) { simSlotNumber++) {
final Preference multiSimPreference = createNewPreference(screen.getContext()); final Preference multiSimPreference = createNewPreference(screen.getContext());
multiSimPreference.setSelectable(false); multiSimPreference.setSelectable(true);
multiSimPreference.setCopyingEnabled(true); multiSimPreference.setCopyingEnabled(true);
multiSimPreference.setOrder(phonePreferenceOrder + simSlotNumber); multiSimPreference.setOrder(phonePreferenceOrder + simSlotNumber);
multiSimPreference.setKey(KEY_PHONE_NUMBER + simSlotNumber); multiSimPreference.setKey(KEY_PHONE_NUMBER + simSlotNumber);
@@ -92,7 +113,7 @@ public class PhoneNumberPreferenceController extends BasePreferenceController {
for (int simSlotNumber = 0; simSlotNumber < mPreferenceList.size(); simSlotNumber++) { for (int simSlotNumber = 0; simSlotNumber < mPreferenceList.size(); simSlotNumber++) {
final Preference simStatusPreference = mPreferenceList.get(simSlotNumber); final Preference simStatusPreference = mPreferenceList.get(simSlotNumber);
simStatusPreference.setTitle(getPreferenceTitle(simSlotNumber)); simStatusPreference.setTitle(getPreferenceTitle(simSlotNumber));
setPhoneNumber(simSlotNumber); simStatusPreference.setSummary(getSummary());
} }
} }
@@ -133,7 +154,7 @@ public class PhoneNumberPreferenceController extends BasePreferenceController {
} }
@VisibleForTesting @VisibleForTesting
protected String getFormattedPhoneNumber(SubscriptionInfo subscriptionInfo) { protected CharSequence getFormattedPhoneNumber(SubscriptionInfo subscriptionInfo) {
final String phoneNumber = SubscriptionUtil.getBidiFormattedPhoneNumber(mContext, final String phoneNumber = SubscriptionUtil.getBidiFormattedPhoneNumber(mContext,
subscriptionInfo); subscriptionInfo);
return TextUtils.isEmpty(phoneNumber) ? mContext.getString(R.string.device_info_default) return TextUtils.isEmpty(phoneNumber) ? mContext.getString(R.string.device_info_default)
@@ -142,6 +163,6 @@ public class PhoneNumberPreferenceController extends BasePreferenceController {
@VisibleForTesting @VisibleForTesting
protected Preference createNewPreference(Context context) { protected Preference createNewPreference(Context context) {
return new Preference(context); return new PhoneNumberSummaryPreference(context);
} }
} }

View File

@@ -33,6 +33,7 @@ import androidx.preference.PreferenceScreen;
import com.android.settings.R; import com.android.settings.R;
import com.android.settings.core.BasePreferenceController; import com.android.settings.core.BasePreferenceController;
import com.android.settings.deviceinfo.PhoneNumberSummaryPreference;
import com.android.settings.deviceinfo.simstatus.SlotSimStatus; import com.android.settings.deviceinfo.simstatus.SlotSimStatus;
import com.android.settings.network.SubscriptionUtil; import com.android.settings.network.SubscriptionUtil;
import com.android.settingslib.Utils; import com.android.settingslib.Utils;
@@ -112,6 +113,11 @@ public class ImeiInfoPreferenceController extends BasePreferenceController {
updatePreference(preference, keyToSlotIndex(preference.getKey())); updatePreference(preference, keyToSlotIndex(preference.getKey()));
} }
@Override
public CharSequence getSummary() {
return mContext.getString(R.string.device_info_protected_single_press);
}
private CharSequence getSummary(int simSlot) { private CharSequence getSummary(int simSlot) {
final int phoneType = getPhoneType(simSlot); final int phoneType = getPhoneType(simSlot);
return phoneType == PHONE_TYPE_CDMA ? mTelephonyManager.getMeid(simSlot) return phoneType == PHONE_TYPE_CDMA ? mTelephonyManager.getMeid(simSlot)
@@ -148,12 +154,8 @@ public class ImeiInfoPreferenceController extends BasePreferenceController {
@VisibleForTesting @VisibleForTesting
protected void updatePreference(Preference preference, int simSlot) { protected void updatePreference(Preference preference, int simSlot) {
if (simSlot < 0) {
preference.setVisible(false);
return;
}
preference.setTitle(getTitle(simSlot)); preference.setTitle(getTitle(simSlot));
preference.setSummary(getSummary(simSlot)); preference.setSummary(getSummary());
} }
private CharSequence getTitleForGsmPhone(int simSlot, boolean isPrimaryImei) { private CharSequence getTitleForGsmPhone(int simSlot, boolean isPrimaryImei) {
@@ -197,6 +199,6 @@ public class ImeiInfoPreferenceController extends BasePreferenceController {
@VisibleForTesting @VisibleForTesting
Preference createNewPreference(Context context) { Preference createNewPreference(Context context) {
return new Preference(context); return new PhoneNumberSummaryPreference(context);
} }
} }

View File

@@ -96,7 +96,6 @@ class SimEidPreferenceController(context: Context, preferenceKey: String) :
} }
preference.title = title preference.title = title
preference.dialogTitle = title preference.dialogTitle = title
preference.summary = eid
updateDialog() updateDialog()
} }
} }
@@ -134,6 +133,9 @@ class SimEidPreferenceController(context: Context, preferenceKey: String) :
val qrCodeView = dialog.requireViewById<ImageView>(R.id.esim_id_qrcode) val qrCodeView = dialog.requireViewById<ImageView>(R.id.esim_id_qrcode)
qrCodeView.setImageBitmap(getEidQrCode(eid)) qrCodeView.setImageBitmap(getEidQrCode(eid))
// After "Tap to show", eid is displayed on preference.
preference.summary = textView.text
} }
override fun handlePreferenceTreeClick(preference: Preference): Boolean { override fun handlePreferenceTreeClick(preference: Preference): Boolean {