Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Ensure the learn more string meet Accessibility requirement." into tm-dev am: 53cd248506 am: fc37e3e6a2

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/18089226

Change-Id: I170b4ce98cbc5a07c79ab03e0ddc531923185db5
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Edgar Wang
2022-05-20 23:52:44 +00:00
committed by Automerger Merge Worker
parent 541191d6c6 fc37e3e6a2
commit 1cde9662b4
2 changed files with 11 additions and 44 deletions
Download Patch File Download Diff File Expand all files Collapse all files

48
packages/SettingsLib/FooterPreference/src/com/android/settingslib/widget/FooterPreference.java
View File

@@ -44,7 +44,6 @@ public class FooterPreference extends Preference {
int mIconVisibility = View.VISIBLE;
private CharSequence mContentDescription;
private CharSequence mLearnMoreText;
private CharSequence mLearnMoreContentDescription;
private FooterLearnMoreSpan mLearnMoreSpan;
public FooterPreference(Context context, AttributeSet attrs) {
@@ -80,9 +79,6 @@ public class FooterPreference extends Preference {
learnMoreText.setSpan(mLearnMoreSpan, 0,
learnMoreText.length(), 0);
learnMore.setText(learnMoreText);
if (!TextUtils.isEmpty(mLearnMoreContentDescription)) {
learnMore.setContentDescription(mLearnMoreContentDescription);
}
} else {
learnMore.setVisibility(View.GONE);
}
@@ -139,27 +135,6 @@ public class FooterPreference extends Preference {
}
}
/**
* To set content description of the learn more text. This can use for talkback
* environment if developer wants to have a customization content.
*
* @param learnMoreContentDescription The resource id of the content description.
*/
public void setLearnMoreContentDescription(CharSequence learnMoreContentDescription) {
if (!TextUtils.equals(mContentDescription, learnMoreContentDescription)) {
mLearnMoreContentDescription = learnMoreContentDescription;
notifyChanged();
}
}
/**
* Return the content description of learn more link.
*/
@VisibleForTesting
CharSequence getLearnMoreContentDescription() {
return mLearnMoreContentDescription;
}
/**
* Assign an action for the learn more link.
*/
@@ -201,7 +176,7 @@ public class FooterPreference extends Preference {
private String mKey;
private CharSequence mTitle;
private CharSequence mContentDescription;
private CharSequence mLearnMoreContentDescription;
private CharSequence mLearnMoreText;
public Builder(@NonNull Context context) {
mContext = context;
@@ -260,25 +235,24 @@ public class FooterPreference extends Preference {
}
/**
* To set content description of the learn more text. This can use for talkback
* To set learn more string of the learn more text. This can use for talkback
* environment if developer wants to have a customization content.
*
* @param learnMoreContentDescription The resource id of the content description.
* @param learnMoreText The resource id of the learn more string.
*/
public Builder setLearnMoreContentDescription(CharSequence learnMoreContentDescription) {
mLearnMoreContentDescription = learnMoreContentDescription;
public Builder setLearnMoreText(CharSequence learnMoreText) {
mLearnMoreText = learnMoreText;
return this;
}
/**
* To set content description of the {@link FooterPreference}. This can use for talkback
* To set learn more string of the {@link FooterPreference}. This can use for talkback
* environment if developer wants to have a customization content.
*
* @param learnMoreContentDescriptionResId The resource id of the content description.
* @param learnMoreTextResId The resource id of the learn more string.
*/
public Builder setLearnMoreContentDescription(
@StringRes int learnMoreContentDescriptionResId) {
mLearnMoreContentDescription = mContext.getText(learnMoreContentDescriptionResId);
public Builder setLearnMoreText(@StringRes int learnMoreTextResId) {
mLearnMoreText = mContext.getText(learnMoreTextResId);
return this;
}
@@ -301,8 +275,8 @@ public class FooterPreference extends Preference {
footerPreference.setContentDescription(mContentDescription);
}
if (!TextUtils.isEmpty(mLearnMoreContentDescription)) {
footerPreference.setLearnMoreContentDescription(mLearnMoreContentDescription);
if (!TextUtils.isEmpty(mLearnMoreText)) {
footerPreference.setLearnMoreText(mLearnMoreText);
}
return footerPreference;
}

7
packages/SettingsLib/tests/robotests/src/com/android/settingslib/widget/FooterPreferenceTest.java
View File

@@ -73,13 +73,6 @@ public class FooterPreferenceTest {
assertThat(mFooterPreference.getContentDescription()).isEqualTo("test");
}
@Test
public void setLearnMoreContentDescription_contentSet_shouldGetSameContentDescription() {
mFooterPreference.setLearnMoreContentDescription("test");
assertThat(mFooterPreference.getLearnMoreContentDescription()).isEqualTo("test");
}
@Test
public void setLearnMoreAction_actionSet_shouldGetAction() {
mFooterPreference.setLearnMoreAction(v -> {