New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 677147 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Jan 14
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug
Team-Accessibility

Blocked on:
issue 707194



Sign in to add a comment

aria-describedby not read if input type text has a placeholder

Project Member Reported by chaok@google.com, Dec 27 2016

Issue description

I am having an issue with aria-deacribedby being read for android talkback
on the chrome browser when the user focuss on the field.

It seems to work for an input type text without a placeholder but does not
read it if the input has a placeholder text or is a select control.

It work on iOS , jaws and NVDA, is there any way I can fix this for android
chrome witouth breaking it for the others.

Here is a an example of a working field and 2  problem fields.

http://output.jsbin.com/tozacusoci

If the link does not work below is the code snippet

    <strong>Works</strong>
    <div>
      <label for='username' >username</label>
      <input id='username' type="text" aria-describedby="worksError"
aria-invalid="true"/>
      <div id="worksError" class="errorText" aria-live="polite">error
please enter a value</div>
    </div>


    <strong>does not work</strong>
    <div>
      <label for='email'>Email</label>
      <input id='email' type="text" placeholder="JohnDoe@gmail.com"
aria-describedby="emailError" aria-invalid="true"/>
      <div id="emailError" class="errorText" aria-live="polite">error
please fill in a valid username</div>
    </div>

    <div>
      <label for='year'>Year</label>
      <select id='year' aria-describedby="yearError" aria-invalid="true">
        <option value="">select year</option>
        <option value="2016">2016</option>
        <option value="2017">2017</option>
      </select>
      <div id="yearError" class="errorText" aria-live="polite">error please
select a year</div>
    </div>
 
Status: Assigned (was: Untriaged)
Labels: NewComponent-Accessibility-Blink
Labels: NewComponent-Accessibility
Components: Blink>Accessibility
Components: -UI>Accessibility
Labels: -newcomponent-accessibility-blink -newcomponent-accessibility
Labels: triage-alice
Labels: triage-dominic-android
Blockedon: 707194
Labels: -Pri-3 -triage-alice -triage-dominic-android Pri-2
NextAction: ----
Owner: ----
Status: Available (was: Assigned)
We need to re-investigate after the HINT support lands.

Labels: triage-dougt
Labels: -triage-dougt
Project Member

Comment 11 by sheriffbot@chromium.org, Aug 13

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Pri-2 Pri-3
Status: Available (was: Untriaged)
Related to issue 713600 -- using a separate placeholder mapping. That issue is on hold. Keeping this as P3 Available and will contact AT vendors about supporting placeholder mapping.
Labels: a11y-required
Owner: dmazz...@chromium.org
Status: Assigned (was: Available)
The behavior now is that aria-describedby is overriding the placeholder. We should fix that.

Project Member

Comment 16 by bugdroid1@chromium.org, Jan 11

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b2cb7a66f4df54ce67c2d282b2ae2353d9d9ded4

commit b2cb7a66f4df54ce67c2d282b2ae2353d9d9ded4
Author: Dominic Mazzoni <dmazzoni@chromium.org>
Date: Fri Jan 11 21:13:37 2019

Android accessible hint text should include placeholder.

Previously a text field's placeholder was exposed as the
hint in many circumstances, but a field that had
aria-describedby would override the placeholder.

Update the logic so that the placeholder text is
always included in the hint text for a node.

Bug:  677147 
Change-Id: I15c7cfede914db6096d88039213793ec2eaa8ad4
Reviewed-on: https://chromium-review.googlesource.com/c/1385724
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: Aaron Leventhal <aleventhal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#622150}
[modify] https://crrev.com/b2cb7a66f4df54ce67c2d282b2ae2353d9d9ded4/content/browser/accessibility/browser_accessibility_android.cc
[modify] https://crrev.com/b2cb7a66f4df54ce67c2d282b2ae2353d9d9ded4/content/test/data/accessibility/aria/input-text-aria-placeholder-expected-android.txt
[modify] https://crrev.com/b2cb7a66f4df54ce67c2d282b2ae2353d9d9ded4/content/test/data/accessibility/html/input-text-name-calc-expected-android.txt
[modify] https://crrev.com/b2cb7a66f4df54ce67c2d282b2ae2353d9d9ded4/content/test/data/accessibility/html/input-text-name-calc-expected-auralinux.txt
[modify] https://crrev.com/b2cb7a66f4df54ce67c2d282b2ae2353d9d9ded4/content/test/data/accessibility/html/input-text-name-calc-expected-blink.txt
[modify] https://crrev.com/b2cb7a66f4df54ce67c2d282b2ae2353d9d9ded4/content/test/data/accessibility/html/input-text-name-calc-expected-win.txt
[modify] https://crrev.com/b2cb7a66f4df54ce67c2d282b2ae2353d9d9ded4/content/test/data/accessibility/html/input-text-name-calc.html

Status: Fixed (was: Started)

Sign in to add a comment