Commit 9a1a93c6 authored by Ian Wells's avatar Ian Wells Committed by Commit Bot

Ensure snackbar action button text is spoken by TalkBack

The spoken message for snackbars now sounds like "Snackbar message. <pause> Action. <pause> Options near the bottom of the screen."

Bug: 1103062
Change-Id: If3747989e82d34d19d1505fbf86aa1e2f967d64f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2342054Reviewed-by: default avatarPavel Yatsuk <pavely@chromium.org>
Commit-Queue: Ian Wells <iwells@chromium.org>
Cr-Commit-Position: refs/heads/master@{#795999}
parent 1fc6c072
......@@ -205,7 +205,8 @@ public class SnackbarView {
* the mMessageView content description is read aloud if accessibility is enabled.
*/
public void announceforAccessibility() {
mMessageView.announceForAccessibility(mMessageView.getContentDescription() + " "
mMessageView.announceForAccessibility(mMessageView.getContentDescription() + ". "
+ mActionButtonView.getContentDescription() + ". "
+ mContainerView.getResources().getString(R.string.bottom_bar_screen_position));
}
......@@ -274,7 +275,6 @@ public class SnackbarView {
mMessageView.setMaxLines(snackbar.getSingleLine() ? 1 : MAX_LINES);
mMessageView.setTemplate(snackbar.getTemplateText());
setViewText(mMessageView, snackbar.getText(), animate);
String actionText = snackbar.getActionText();
ApiCompatibilityUtils.setTextAppearance(mMessageView, getTextAppearance(snackbar));
ApiCompatibilityUtils.setTextAppearance(
......@@ -291,8 +291,9 @@ public class SnackbarView {
mSnackbarView.setBackgroundColor(backgroundColor);
}
if (actionText != null) {
if (snackbar.getActionText() != null) {
mActionButtonView.setVisibility(View.VISIBLE);
mActionButtonView.setContentDescription(snackbar.getActionText());
setViewText(mActionButtonView, snackbar.getActionText(), animate);
} else {
mActionButtonView.setVisibility(View.GONE);
......
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