Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e6f28fbd authored by Yi Jiang's avatar Yi Jiang Committed by Presubmit Automerger Backend
Browse files

[automerge] Fixes 'learn more' link in ScreenTimeoutSettings 2p: 284cb1c2 

Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Settings/+/17578008

Bug: 227292811
Change-Id: Ibab34cc241b50c473e6951d46f6be1697bfadb03
parents 2ec4bc0b 284cb1c2

src/com/android/settings/display/ScreenTimeoutSettings.java

+6 −25
Original line number Diff line number Diff line
@@ -32,14 +32,8 @@ import android.hardware.SensorPrivacyManager; 
import android.os.PowerManager;

import android.os.UserHandle;

import android.provider.Settings;

import android.text.SpannableString;

import android.text.Spanned;

import android.text.style.ClickableSpan;

import android.util.Log;

import android.view.View;



import androidx.annotation.NonNull;

import androidx.preference.Preference;

import androidx.preference.PreferenceScreen;



import com.android.settings.R;
@@ -95,7 +89,7 @@ public class ScreenTimeoutSettings extends RadioButtonPickerFragment implements 
    RestrictedLockUtils.EnforcedAdmin mAdmin;



    @VisibleForTesting

    Preference mDisableOptionsPreference;

    FooterPreference mDisableOptionsPreference;



    @VisibleForTesting

    AdaptiveSleepPermissionPreferenceController mAdaptiveSleepPermissionController;
@@ -228,26 +222,13 @@ public class ScreenTimeoutSettings extends RadioButtonPickerFragment implements 
                        R.string.admin_disabled_other_options));

        final String textMoreDetails = getResources().getString(R.string.admin_more_details);



        final SpannableString spannableString = new SpannableString(

                textDisabledByAdmin + System.lineSeparator()

                        + System.lineSeparator() + textMoreDetails);

        final ClickableSpan clickableSpan = new ClickableSpan() {

            @Override

            public void onClick(@NonNull View widget) {

                RestrictedLockUtils.sendShowAdminSupportDetailsIntent(getContext(), mAdmin);

            }

        };



        if (textDisabledByAdmin != null && textMoreDetails != null) {

            spannableString.setSpan(clickableSpan, textDisabledByAdmin.length() + 1,

                    textDisabledByAdmin.length() + textMoreDetails.length() + 2,

                    Spanned.SPAN_EXCLUSIVE_EXCLUSIVE);

        }



        mDisableOptionsPreference = new FooterPreference(getContext());

        mDisableOptionsPreference.setLayoutResource(R.layout.preference_footer);

        mDisableOptionsPreference.setTitle(spannableString);

        mDisableOptionsPreference.setTitle(textDisabledByAdmin);

        mDisableOptionsPreference.setSelectable(false);

        mDisableOptionsPreference.setLearnMoreText(textMoreDetails);

        mDisableOptionsPreference.setLearnMoreAction(v -> {

            RestrictedLockUtils.sendShowAdminSupportDetailsIntent(getContext(), mAdmin);

        });

        mDisableOptionsPreference.setIcon(R.drawable.ic_info_outline_24dp);



        // The 'disabled by admin' preference should always be at the end of the setting page.

tests/robotests/src/com/android/settings/display/ScreenTimeoutSettingsTest.java

+2 −2
Original line number Diff line number Diff line
@@ -44,12 +44,12 @@ import android.content.res.Resources; 
import android.provider.SearchIndexableResource;

import android.provider.Settings;



import androidx.preference.Preference;

import androidx.preference.PreferenceScreen;



import com.android.settings.R;

import com.android.settings.testutils.FakeFeatureFactory;

import com.android.settingslib.RestrictedLockUtils;

import com.android.settingslib.widget.FooterPreference;



import org.junit.Before;

import org.junit.Test;
@@ -86,7 +86,7 @@ public class ScreenTimeoutSettingsTest { 
    AdaptiveSleepBatterySaverPreferenceController mAdaptiveSleepBatterySaverPreferenceController;



    @Mock

    Preference mDisableOptionsPreference;

    FooterPreference mDisableOptionsPreference;



    @Mock

    private PackageManager mPackageManager;