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

Issue 792056 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Dec 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug
Team-Accessibility



Sign in to add a comment

Some dropdown boxes don't have options read with Jaws 18

Project Member Reported by lpalmaro@chromium.org, Dec 5 2017

Issue description

OS: Windows 

Reported feedback from Googler, anecdotally reproducing across Chrome versions. 

Some drop down boxes don’t have their options read using Jaws 18. See the www.kayak.com site as one example. When you get to the number of passengers, the experience is a mess.
 
Owner: dsexton@chromium.org
Adding David as owner for further investigation
Owner: ----
This appears to be related to spin box controls, see also  bug 793113 
JAWS traditionally works well with these, but on this particular site we need a more technical evaluation of the issue.
Owner: nek...@chromium.org
Status: Assigned (was: Available)
Assigning to Nektarios per his direction.
Project Member

Comment 4 by bugdroid1@chromium.org, Dec 14 2017

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

commit a592d0fa16a10907e831fe30076cba31fb848d25
Author: Nektarios Paisios <nektar@chromium.org>
Date: Thu Dec 14 21:38:35 2017

Exposed the aria-valuenow attribute for ARIA-based spin buttons

1. Stopped using the value attribute to hold the inner text of spin buttons and other range controls because this could result in showing the inner text as the control's value which, depending on the control's author, could be totally meaningless. For spin buttons, now the value attribute only holds the contents of aria-valuetext if present.
2. Took the opportunity to write some more tests and clean up some of our Blink code.
R=dmazzoni@chromium.org, aleventhal@chromium.org

Also: 
Bug:  792056 
Change-Id: Ib4eb98c2c1a351f40e2ac144b1967142040f0675
Tested: manually with dropdowns on kayak.com, browser tests, sample HTML snippet of an ARIA spin button
Reviewed-on: https://chromium-review.googlesource.com/823136
Commit-Queue: Nektarios Paisios <nektar@chromium.org>
Reviewed-by: Aaron Leventhal <aleventhal@chromium.org>
Reviewed-by: Nektarios Paisios <nektar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#524190}
[modify] https://crrev.com/a592d0fa16a10907e831fe30076cba31fb848d25/content/browser/accessibility/dump_accessibility_tree_browsertest.cc
[modify] https://crrev.com/a592d0fa16a10907e831fe30076cba31fb848d25/content/test/data/accessibility/aria/aria-readonly-expected-android.txt
[modify] https://crrev.com/a592d0fa16a10907e831fe30076cba31fb848d25/content/test/data/accessibility/aria/aria-spinbutton-expected-android.txt
[modify] https://crrev.com/a592d0fa16a10907e831fe30076cba31fb848d25/content/test/data/accessibility/aria/aria-spinbutton-expected-mac.txt
[modify] https://crrev.com/a592d0fa16a10907e831fe30076cba31fb848d25/content/test/data/accessibility/aria/aria-spinbutton-expected-win.txt
[modify] https://crrev.com/a592d0fa16a10907e831fe30076cba31fb848d25/content/test/data/accessibility/aria/aria-spinbutton.html
[add] https://crrev.com/a592d0fa16a10907e831fe30076cba31fb848d25/content/test/data/accessibility/html/input-number-expected-android.txt
[add] https://crrev.com/a592d0fa16a10907e831fe30076cba31fb848d25/content/test/data/accessibility/html/input-number-expected-blink.txt
[add] https://crrev.com/a592d0fa16a10907e831fe30076cba31fb848d25/content/test/data/accessibility/html/input-number-expected-mac.txt
[add] https://crrev.com/a592d0fa16a10907e831fe30076cba31fb848d25/content/test/data/accessibility/html/input-number-expected-win.txt
[add] https://crrev.com/a592d0fa16a10907e831fe30076cba31fb848d25/content/test/data/accessibility/html/input-number.html
[modify] https://crrev.com/a592d0fa16a10907e831fe30076cba31fb848d25/third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp
[modify] https://crrev.com/a592d0fa16a10907e831fe30076cba31fb848d25/ui/accessibility/platform/ax_platform_node_win.cc

Comment 5 by nek...@chromium.org, Dec 15 2017

Status: Fixed (was: Assigned)
Status: Verified (was: Fixed)

Sign in to add a comment