From 010306644b6a07469bc0ff1094c26ab337bad8ad Mon Sep 17 00:00:00 2001 From: Bonian Chen Date: Mon, 29 Nov 2021 12:06:38 +0800 Subject: [PATCH] [Settings] Refactor Wifi Calling description text Refactor the display of description text when no wifi calling options available. Enhancement: 1. These text need to align the style with the summary part of the wifi calling options. 2. These text need to be scrollable in order to see full description. 3. The link within text need to be clickable. Bug: 204844012 Test: local Change-Id: Ie89a5b65e75cdd61fab68c6e1bf7c40bfcc68b6b --- .../wifi_calling_settings_preferences.xml | 9 +-- res/xml/wifi_calling_settings.xml | 7 ++ .../calling/LinkifyDescriptionPreference.java | 70 +++++++++++++++++++ .../calling/WifiCallingSettingsForSub.java | 68 +++++++++--------- 4 files changed, 115 insertions(+), 39 deletions(-) create mode 100644 src/com/android/settings/wifi/calling/LinkifyDescriptionPreference.java diff --git a/res/layout/wifi_calling_settings_preferences.xml b/res/layout/wifi_calling_settings_preferences.xml index 98acd9525a3..9a6cbe6698b 100644 --- a/res/layout/wifi_calling_settings_preferences.xml +++ b/res/layout/wifi_calling_settings_preferences.xml @@ -29,16 +29,11 @@ - - diff --git a/res/xml/wifi_calling_settings.xml b/res/xml/wifi_calling_settings.xml index 0276bdb6b10..902ff1af2fa 100644 --- a/res/xml/wifi_calling_settings.xml +++ b/res/xml/wifi_calling_settings.xml @@ -21,6 +21,7 @@ + + diff --git a/src/com/android/settings/wifi/calling/LinkifyDescriptionPreference.java b/src/com/android/settings/wifi/calling/LinkifyDescriptionPreference.java new file mode 100644 index 00000000000..60400b0f354 --- /dev/null +++ b/src/com/android/settings/wifi/calling/LinkifyDescriptionPreference.java @@ -0,0 +1,70 @@ +/* + * Copyright (C) 2021 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package com.android.settings.wifi.calling; + +import android.content.Context; +import android.text.SpannableString; +import android.text.TextUtils; +import android.text.method.LinkMovementMethod; +import android.text.style.ClickableSpan; +import android.text.util.Linkify; +import android.util.AttributeSet; +import android.view.View; +import android.widget.TextView; + +import androidx.core.text.util.LinkifyCompat; +import androidx.preference.Preference; +import androidx.preference.PreferenceViewHolder; + +import com.android.settings.R; + +/** A preference which supports linkify text as a description in the summary **/ +public class LinkifyDescriptionPreference extends Preference { + + public LinkifyDescriptionPreference(Context context) { + this(context, null); + } + + public LinkifyDescriptionPreference(Context context, AttributeSet attrs) { + super(context, attrs); + } + + @Override + public void onBindViewHolder(PreferenceViewHolder holder) { + super.onBindViewHolder(holder); + + final TextView summaryView = (TextView) holder.findViewById(android.R.id.summary); + if (summaryView == null || summaryView.getVisibility() != View.VISIBLE) { + return; + } + + final CharSequence summary = getSummary(); + if (TextUtils.isEmpty(summary)) { + return; + } + + summaryView.setMaxLines(Integer.MAX_VALUE); + + final SpannableString spannableSummary = new SpannableString(summary); + if (spannableSummary.getSpans(0, spannableSummary.length(), ClickableSpan.class) + .length > 0) { + summaryView.setMovementMethod(LinkMovementMethod.getInstance()); + } + LinkifyCompat.addLinks(summaryView, + Linkify.WEB_URLS | Linkify.EMAIL_ADDRESSES | Linkify.PHONE_NUMBERS); + } +} diff --git a/src/com/android/settings/wifi/calling/WifiCallingSettingsForSub.java b/src/com/android/settings/wifi/calling/WifiCallingSettingsForSub.java index 18583f1ae82..19664be4094 100644 --- a/src/com/android/settings/wifi/calling/WifiCallingSettingsForSub.java +++ b/src/com/android/settings/wifi/calling/WifiCallingSettingsForSub.java @@ -35,7 +35,6 @@ import android.telephony.ims.ImsManager; import android.telephony.ims.ImsMmTelManager; import android.telephony.ims.ProvisioningManager; import android.text.TextUtils; -import android.text.util.Linkify; import android.util.Log; import android.view.LayoutInflater; import android.view.View; @@ -58,8 +57,11 @@ import com.android.settings.Utils; import com.android.settings.core.SubSettingLauncher; import com.android.settings.network.ims.WifiCallingQueryImsState; import com.android.settings.widget.SettingsMainSwitchBar; +import com.android.settings.wifi.calling.LinkifyDescriptionPreference; import com.android.settingslib.widget.OnMainSwitchChangeListener; +import java.util.List; + /** * This is the inner class of {@link WifiCallingSettings} fragment. * The preference screen lets you enable/disable Wi-Fi Calling and change Wi-Fi Calling mode. @@ -73,6 +75,7 @@ public class WifiCallingSettingsForSub extends SettingsPreferenceFragment private static final String BUTTON_WFC_MODE = "wifi_calling_mode"; private static final String BUTTON_WFC_ROAMING_MODE = "wifi_calling_roaming_mode"; private static final String PREFERENCE_EMERGENCY_ADDRESS = "emergency_address_key"; + private static final String PREFERENCE_NO_OPTIONS_DESC = "no_options_description"; @VisibleForTesting static final int REQUEST_CHECK_WFC_EMERGENCY_ADDRESS = 1; @@ -92,7 +95,6 @@ public class WifiCallingSettingsForSub extends SettingsPreferenceFragment private ListWithEntrySummaryPreference mButtonWfcMode; private ListWithEntrySummaryPreference mButtonWfcRoamingMode; private Preference mUpdateAddress; - private TextView mEmptyView; private boolean mValidListener = false; private boolean mEditableWfcMode = true; @@ -157,7 +159,7 @@ public class WifiCallingSettingsForSub extends SettingsPreferenceFragment } /* - * Launch carrier emergency address managemnent activity + * Launch carrier emergency address management activity */ private final OnPreferenceClickListener mUpdateAddressListener = preference -> { @@ -186,14 +188,6 @@ public class WifiCallingSettingsForSub extends SettingsPreferenceFragment public void onActivityCreated(Bundle savedInstanceState) { super.onActivityCreated(savedInstanceState); - mEmptyView = getView().findViewById(android.R.id.empty); - setEmptyView(mEmptyView); - mEmptyView.setAutoLinkMask(Linkify.WEB_URLS); - final Resources res = getResourcesForSubId(); - final String emptyViewText = res.getString(R.string.wifi_calling_off_explanation, - res.getString(R.string.wifi_calling_off_explanation_2)); - mEmptyView.setText(emptyViewText); - mSwitchBar = getView().findViewById(R.id.switch_bar); mSwitchBar.show(); } @@ -309,6 +303,9 @@ public class WifiCallingSettingsForSub extends SettingsPreferenceFragment mIntentFilter = new IntentFilter(); mIntentFilter.addAction(ImsManager.ACTION_WFC_IMS_REGISTRATION_ERROR); + + updateDescriptionForOptions( + List.of(mButtonWfcMode, mButtonWfcRoamingMode, mUpdateAddress)); } @Override @@ -324,7 +321,7 @@ public class WifiCallingSettingsForSub extends SettingsPreferenceFragment final View view = inflater.inflate( R.layout.wifi_calling_settings_preferences, container, false); - final ViewGroup prefs_container = view.findViewById(R.id.prefs_container); + final ViewGroup prefs_container = view.findViewById(android.R.id.tabcontent); Utils.prepareCustomPreferencesList(container, view, prefs_container, false); final View prefs = super.onCreateView(inflater, prefs_container, savedInstanceState); prefs_container.addView(prefs); @@ -573,28 +570,35 @@ public class WifiCallingSettingsForSub extends SettingsPreferenceFragment final PreferenceScreen preferenceScreen = getPreferenceScreen(); final boolean updateAddressEnabled = (getCarrierActivityIntent() != null); if (wfcEnabled) { - if (mEditableWfcMode) { - preferenceScreen.addPreference(mButtonWfcMode); - } else { - // Don't show WFC (home) preference if it's not editable. - preferenceScreen.removePreference(mButtonWfcMode); - } - if (mEditableWfcRoamingMode && !mUseWfcHomeModeForRoaming) { - preferenceScreen.addPreference(mButtonWfcRoamingMode); - } else { - // Don't show WFC roaming preference if it's not editable. - preferenceScreen.removePreference(mButtonWfcRoamingMode); - } - if (updateAddressEnabled) { - preferenceScreen.addPreference(mUpdateAddress); - } else { - preferenceScreen.removePreference(mUpdateAddress); - } + // Don't show WFC (home) preference if it's not editable. + mButtonWfcMode.setVisible(mEditableWfcMode); + // Don't show WFC roaming preference if it's not editable. + mButtonWfcRoamingMode.setVisible( + mEditableWfcRoamingMode && !mUseWfcHomeModeForRoaming); + mUpdateAddress.setVisible(updateAddressEnabled); } else { - preferenceScreen.removePreference(mButtonWfcMode); - preferenceScreen.removePreference(mButtonWfcRoamingMode); - preferenceScreen.removePreference(mUpdateAddress); + mButtonWfcMode.setVisible(false); + mButtonWfcRoamingMode.setVisible(false); + mUpdateAddress.setVisible(false); } + updateDescriptionForOptions( + List.of(mButtonWfcMode, mButtonWfcRoamingMode, mUpdateAddress)); + } + + private void updateDescriptionForOptions(List visibleOptions) { + LinkifyDescriptionPreference pref = findPreference(PREFERENCE_NO_OPTIONS_DESC); + if (pref == null) { + return; + } + + boolean optionsAvailable = visibleOptions.stream().anyMatch(Preference::isVisible); + if (!optionsAvailable) { + final Resources res = getResourcesForSubId(); + String emptyViewText = res.getString(R.string.wifi_calling_off_explanation, + res.getString(R.string.wifi_calling_off_explanation_2)); + pref.setSummary(emptyViewText); + } + pref.setVisible(!optionsAvailable); } @Override