diff --git a/packages/SettingsLib/res/values/strings.xml b/packages/SettingsLib/res/values/strings.xml index 064cc84dc930f..0b51a8b3e71b5 100644 --- a/packages/SettingsLib/res/values/strings.xml +++ b/packages/SettingsLib/res/values/strings.xml @@ -110,6 +110,9 @@ Very Fast + + %1$s / %2$s + Disconnected diff --git a/packages/SettingsLib/src/com/android/settingslib/wifi/AccessPoint.java b/packages/SettingsLib/src/com/android/settingslib/wifi/AccessPoint.java index ec4141518f827..9a3665f3028f1 100644 --- a/packages/SettingsLib/src/com/android/settingslib/wifi/AccessPoint.java +++ b/packages/SettingsLib/src/com/android/settingslib/wifi/AccessPoint.java @@ -677,13 +677,6 @@ public class AccessPoint implements Comparable { // Update to new summary StringBuilder summary = new StringBuilder(); - // TODO(b/62354743): Standardize and international delimiter usage - final String concatenator = " / "; - - if (mSpeed != Speed.NONE) { - summary.append(getSpeedLabel() + concatenator); - } - if (isActive() && config != null && config.isPasspoint()) { // This is the active connection on passpoint summary.append(getSummary(mContext, getDetailedState(), @@ -767,14 +760,15 @@ public class AccessPoint implements Comparable { } } - // Strip trailing delimiter if applicable - int concatLength = concatenator.length(); - if (summary.length() >= concatLength && summary.substring( - summary.length() - concatLength, summary.length()).equals(concatenator)) { - summary.delete(summary.length() - concatLength, summary.length()); + // If Speed label is present, use the preference combination to prepend it to the summary. + if (mSpeed != Speed.NONE) { + return mContext.getResources().getString( + R.string.preference_summary_default_combination, + getSpeedLabel(), + summary.toString()); + } else { + return summary.toString(); } - - return summary.toString(); } /**