Fixed
Status Update
Comments
ku...@google.com <ku...@google.com>
ku...@google.com <ku...@google.com> #2
Can you provide the below requested information to better understand the issue:
Steps to reproduce
Please provide a sample application or apk to reproduce the issue.
Also kindly mention the steps to be followed for reproducing the issue with the given sample application.
Android bug report
After reproducing the issue, press the volume up, volume down, and power button simultaneously. This will capture a bug report on your device in the “bug reports” directory. Attach the bug report file to this issue.
Alternate method:
After reproducing the issue, navigate to developer settings, ensure ‘USB debugging’ is enabled, then enable ‘Bug report shortcut’. To take bug report, hold the power button and select the ‘Take bug report’ option.
Steps to reproduce
Please provide a sample application or apk to reproduce the issue.
Also kindly mention the steps to be followed for reproducing the issue with the given sample application.
Android bug report
After reproducing the issue, press the volume up, volume down, and power button simultaneously. This will capture a bug report on your device in the “bug reports” directory. Attach the bug report file to this issue.
Alternate method:
After reproducing the issue, navigate to developer settings, ensure ‘USB debugging’ is enabled, then enable ‘Bug report shortcut’. To take bug report, hold the power button and select the ‘Take bug report’ option.
mi...@willowtreeapps.com <mi...@willowtreeapps.com> #3
Steps to reproduce:
Enable talkback from the accessibility settings menu.
Open app from apk.
Tap "AUTHENTICATE WITH BIOMETRICS" button.
A dialog appears.
Highlight "Touch the fingerprint sensor"
Expected:
The following text is read aloud - "Touch the fingerprint sensor."
Actual:
The following text is read aloud - "Help message area."
Enable talkback from the accessibility settings menu.
Open app from apk.
Tap "AUTHENTICATE WITH BIOMETRICS" button.
A dialog appears.
Highlight "Touch the fingerprint sensor"
Expected:
The following text is read aloud - "Touch the fingerprint sensor."
Actual:
The following text is read aloud - "Help message area."
ku...@google.com <ku...@google.com> #4
Thank you for reporting this issue. We have shared this with our product and engineering team and will update this issue with more information as it becomes available.
kc...@google.com <kc...@google.com>
kc...@google.com <kc...@google.com> #5
Ah, this bug can only be fixed from the framework side (and has already been fixed). I'm marking this as infeasible since there is no way but to have the user update to a newer release (Q)
mi...@willowtreeapps.com <mi...@willowtreeapps.com> #6
There are two dialogs, one from the framework and the other from androidx. The androidx one also has the same bug, attached is a video of it. I think this would be a separate fix and would resolve the bug for users that see the androidx dialog.
kc...@google.com <kc...@google.com> #7
Ah sorry, I assumed it was only on API28 from comment #1 . I'll re-open this bug
kc...@google.com <kc...@google.com> #8
I think we just need to do setContentDescription(String) on the hint view whenever the text changes
cu...@google.com <cu...@google.com> #9
Looks like the "Help message area" text is being set as the content description for the TextView in the layout XML, so I don't think calling setContentDescription is the right fix. We should probably do one of two things:
1. Use android:hint instead of android:contentDescription, so that it augments the visible text instead of overriding it
2. Remove the android:contentDescription attribute altogether and let TalkBack just read aloud the visible text
Any thoughts on which of these would be preferable, Kevin?
1. Use android:hint instead of android:contentDescription, so that it augments the visible text instead of overriding it
2. Remove the android:contentDescription attribute altogether and let TalkBack just read aloud the visible text
Any thoughts on which of these would be preferable, Kevin?
cu...@google.com <cu...@google.com> #10
Actually, I take that back. Option 1 doesn't actually work the way I expected (the hint text is read aloud instead of the visible text), so I'll probably just go with option 2.
ap...@google.com <ap...@google.com> #11
Project: platform/frameworks/support
Branch: androidx-master-dev
commit 256a01352379845d17ba8b0198aca2be4ba445bb
Author: Curtis Belmonte <curtislb@google.com>
Date: Wed Jul 03 11:31:14 2019
Remove content description for fingerprint dialog error text
Currently, the fingerprint dialog shown for API < 28 sets a content
description on the error message TextView, causing that to be read aloud
by TalkBack instead of the actual error message. This commit fixes the
issue by simply removing the content description for the error TextView
so that it doesn't override the visible text for accessibility.
Test: Run androidx biometric demo on Pixel 2XL API 26, launch the
fingerprint dialog with TalkBack enabled, and set focus on "Touch the
fingerprint sensor" view. Verify that TalkBack now reads aloud "Touch
the fingerprint sensor" instead of "Help message area". Also trigger
other errors such as "Not recognized" and verify that these are also
read aloud by TalkBack as they occur.
Fixes: 123572331
Change-Id: I11b351e1ab88c828e2114a357850880759e083be
M biometric/res/layout/fingerprint_dialog_layout.xml
https://android-review.googlesource.com/1011377
https://goto.google.com/android-sha1/256a01352379845d17ba8b0198aca2be4ba445bb
Branch: androidx-master-dev
commit 256a01352379845d17ba8b0198aca2be4ba445bb
Author: Curtis Belmonte <curtislb@google.com>
Date: Wed Jul 03 11:31:14 2019
Remove content description for fingerprint dialog error text
Currently, the fingerprint dialog shown for API < 28 sets a content
description on the error message TextView, causing that to be read aloud
by TalkBack instead of the actual error message. This commit fixes the
issue by simply removing the content description for the error TextView
so that it doesn't override the visible text for accessibility.
Test: Run androidx biometric demo on Pixel 2XL API 26, launch the
fingerprint dialog with TalkBack enabled, and set focus on "Touch the
fingerprint sensor" view. Verify that TalkBack now reads aloud "Touch
the fingerprint sensor" instead of "Help message area". Also trigger
other errors such as "Not recognized" and verify that these are also
read aloud by TalkBack as they occur.
Fixes: 123572331
Change-Id: I11b351e1ab88c828e2114a357850880759e083be
M biometric/res/layout/fingerprint_dialog_layout.xml
[Deleted User] <[Deleted User]> #12
I'm using the androidx.biometric 1.0.1
on the motorola moto x4 with Android 9 installed, this doesn't seem to be fixed. Help Message Area is still read out.
Can we please have a look at this again?
on the motorola moto x4 with Android 9 installed, this doesn't seem to be fixed. Help Message Area is still read out.
Can we please have a look at this again?
Description
Version used: 1.0.0-alpha02
Theme used:
Devices/Android versions reproduced on: Nexus 5X API 28 Emulator.
- Relevant code to trigger the issue.
Steps to reproduce:
1. Enable talkback.
2. Navigate to the "Fingerprint Login" dialog.
3. Tap on the "Touch the fingerprint sensor" TextView.
Expected:
The following text is read aloud - "Touch the fingerprint sensor."
Actual:
The following text is read aloud - "Help message area."
See attached video.