Commit e1c8b33b authored by Xing Liu's avatar Xing Liu Committed by Commit Bot

Download location: better accessibility for radio button.

This CL sets a description for accessibility purpose on the radio
button in download preferences.

Bug: 850254
Change-Id: I5c854041e26094dd69cd95e74cacd06cb3eed4f4
Reviewed-on: https://chromium-review.googlesource.com/1103306Reviewed-by: default avatarDavid Trainor <dtrainor@chromium.org>
Commit-Queue: Xing Liu <xingliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#568249}
parent 22579b1f
......@@ -16,8 +16,7 @@
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginStart="10dp"
android:layout_marginEnd="6dp"
android:importantForAccessibility="no" />
android:layout_marginEnd="6dp" />
<LinearLayout
android:layout_width="0dp"
......
......@@ -71,8 +71,16 @@ public class DownloadLocationPreferenceAdapter
TextView summaryText = (TextView) view.findViewById(R.id.description);
if (isEnabled(position)) {
summaryText.setText(DownloadUtils.getStringForAvailableBytes(
getContext(), directoryOption.availableSpace));
String summary = DownloadUtils.getStringForAvailableBytes(
getContext(), directoryOption.availableSpace);
summaryText.setText(summary);
// Build description for accessibility.
StringBuilder accessibilityDescription = new StringBuilder();
accessibilityDescription.append(directoryOption.name);
accessibilityDescription.append(" ");
accessibilityDescription.append(summary);
radioButton.setContentDescription(accessibilityDescription);
} else {
radioButton.setEnabled(false);
titleText.setEnabled(false);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment