Use accessibilityLiveRegion for ConfirmLockPassword/Pattern errorText
- Revert previous "Add announceForAccessibility to ConfirmLockPattern"; - Add android:accessibilityLiveRegion="polite" for errorText in ConfirmLockPassword/Pattern; Bug:20282779 Change-Id: I4b9c0d461d0dd0504bd88a8a2144e09a42d5f64e
This commit is contained in:
@@ -73,6 +73,7 @@
|
|||||||
style="@style/TextAppearance.PasswordEntry"/>
|
style="@style/TextAppearance.PasswordEntry"/>
|
||||||
|
|
||||||
<TextView style="@style/TextAppearance.ConfirmDeviceCredentialsErrorText"
|
<TextView style="@style/TextAppearance.ConfirmDeviceCredentialsErrorText"
|
||||||
|
android:accessibilityLiveRegion="polite"
|
||||||
android:id="@+id/errorText"
|
android:id="@+id/errorText"
|
||||||
android:layout_width="wrap_content"
|
android:layout_width="wrap_content"
|
||||||
android:layout_height="wrap_content"
|
android:layout_height="wrap_content"
|
||||||
|
@@ -88,6 +88,7 @@
|
|||||||
|
|
||||||
<TextView
|
<TextView
|
||||||
style="@style/TextAppearance.ConfirmDeviceCredentialsErrorText"
|
style="@style/TextAppearance.ConfirmDeviceCredentialsErrorText"
|
||||||
|
android:accessibilityLiveRegion="polite"
|
||||||
android:id="@+id/errorText"
|
android:id="@+id/errorText"
|
||||||
android:layout_width="wrap_content"
|
android:layout_width="wrap_content"
|
||||||
android:layout_height="wrap_content"
|
android:layout_height="wrap_content"
|
||||||
|
@@ -68,6 +68,7 @@
|
|||||||
|
|
||||||
<TextView
|
<TextView
|
||||||
style="@style/TextAppearance.ConfirmDeviceCredentialsErrorText"
|
style="@style/TextAppearance.ConfirmDeviceCredentialsErrorText"
|
||||||
|
android:accessibilityLiveRegion="polite"
|
||||||
android:id="@+id/errorText"
|
android:id="@+id/errorText"
|
||||||
android:layout_width="wrap_content"
|
android:layout_width="wrap_content"
|
||||||
android:layout_height="0dp"
|
android:layout_height="0dp"
|
||||||
|
@@ -70,6 +70,7 @@
|
|||||||
|
|
||||||
<TextView
|
<TextView
|
||||||
style="@style/TextAppearance.ConfirmDeviceCredentialsErrorText"
|
style="@style/TextAppearance.ConfirmDeviceCredentialsErrorText"
|
||||||
|
android:accessibilityLiveRegion="polite"
|
||||||
android:id="@+id/errorText"
|
android:id="@+id/errorText"
|
||||||
android:layout_width="wrap_content"
|
android:layout_width="wrap_content"
|
||||||
android:layout_height="0dp"
|
android:layout_height="0dp"
|
||||||
|
@@ -377,7 +377,6 @@ public class ConfirmLockPassword extends ConfirmDeviceCredentialBaseActivity {
|
|||||||
|
|
||||||
private void showError(CharSequence msg, long timeout) {
|
private void showError(CharSequence msg, long timeout) {
|
||||||
mErrorTextView.setText(msg);
|
mErrorTextView.setText(msg);
|
||||||
mErrorTextView.announceForAccessibility(mErrorTextView.getText());
|
|
||||||
mPasswordEntry.setText(null);
|
mPasswordEntry.setText(null);
|
||||||
mHandler.removeCallbacks(mResetErrorRunnable);
|
mHandler.removeCallbacks(mResetErrorRunnable);
|
||||||
if (timeout != 0) {
|
if (timeout != 0) {
|
||||||
|
@@ -208,7 +208,7 @@ public class ConfirmLockPattern extends ConfirmDeviceCredentialBaseActivity {
|
|||||||
mLockPatternView.clearPattern();
|
mLockPatternView.clearPattern();
|
||||||
break;
|
break;
|
||||||
case NeedToUnlockWrong:
|
case NeedToUnlockWrong:
|
||||||
showError(R.string.lockpattern_need_to_unlock_wrong);
|
mErrorTextView.setText(R.string.lockpattern_need_to_unlock_wrong);
|
||||||
|
|
||||||
mLockPatternView.setDisplayMode(LockPatternView.DisplayMode.Wrong);
|
mLockPatternView.setDisplayMode(LockPatternView.DisplayMode.Wrong);
|
||||||
mLockPatternView.setEnabled(true);
|
mLockPatternView.setEnabled(true);
|
||||||
@@ -382,7 +382,7 @@ public class ConfirmLockPattern extends ConfirmDeviceCredentialBaseActivity {
|
|||||||
@Override
|
@Override
|
||||||
public void onTick(long millisUntilFinished) {
|
public void onTick(long millisUntilFinished) {
|
||||||
final int secondsCountdown = (int) (millisUntilFinished / 1000);
|
final int secondsCountdown = (int) (millisUntilFinished / 1000);
|
||||||
showError(getString(
|
mErrorTextView.setText(getString(
|
||||||
R.string.lockpattern_too_many_failed_confirmation_attempts,
|
R.string.lockpattern_too_many_failed_confirmation_attempts,
|
||||||
secondsCountdown));
|
secondsCountdown));
|
||||||
}
|
}
|
||||||
@@ -394,14 +394,5 @@ public class ConfirmLockPattern extends ConfirmDeviceCredentialBaseActivity {
|
|||||||
}
|
}
|
||||||
}.start();
|
}.start();
|
||||||
}
|
}
|
||||||
|
|
||||||
private void showError(CharSequence msg) {
|
|
||||||
mErrorTextView.setText(msg);
|
|
||||||
mErrorTextView.announceForAccessibility(mErrorTextView.getText());
|
|
||||||
}
|
|
||||||
|
|
||||||
private void showError(int msgid) {
|
|
||||||
showError(getText(msgid));
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user