New issue
Advanced search Search tips

Issue 625411 link

Starred by 4 users
Status: Verified
Owner: ----
Closed: Jul 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug
Team-Accessibility



Sign in to add a comment

Accessible BLockly Doesn't Work in ChromeVox Next

Project Member Reported by chaok@google.com, Jul 2 2016 
Mode: force_next
Version: 53.0.2773.0
Reproduction Steps: 
1. Enable ChromeVOx Next
2. https://accessible-blockly.appspot.com/demos/accessible/index.html
3. TAB to "toolbox, group"
4. UP, DOWN, RIGHT, or LEFT ARROW
5. TAB to "Clear WOrkspace, button"

Observed:
4. Silence
5. "disabled" isn't uttered

Expected:
4. Traverse Accessible BLockly toolbox tree (e.g. category, blocks, actions, etc.)
5. aria-disabled="true" to be reported

Note:
Currently, we're recommending NVDA+FIrefox as the best accessible solution for Accessible Blockly. HOwever, it'd be great to also say that cvox2 is supported.
Project Member

Comment 1 by sheriffbot@chromium.org, Jul 3 2016

Labels: Hotlist-Google

Comment 2 by chaok@google.com, Jul 14 2016 

Please close this bug. cvox2 works with Accessible BLockly now.
Thanks!

Comment 3 by lpalmaro@chromium.org, Jul 15 2016

Labels: phase4
Status: Available (was: Unconfirmed)

Comment 4 by chaok@google.com, Jul 23 2016 

Please mark as fixed+ erified (i.e. close)

Comment 5 by nek...@chromium.org, Jul 25 2016

Status: Verified (was: Available)
Project Member

Comment 6 by bugdroid1@chromium.org, Jul 26 2016 

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

commit 545a777dfe3e6410be16517f78f24602a64a55b6
Author: dtseng <dtseng@chromium.org>
Date: Tue Jul 26 22:47:04 2016

Less strict checked state changes and introduce disabled state for content

Sometimes, ChromeVox places range on the ancestor of a node that receives checked state changed. Ensure we cover this case when we process a checked state changed event.

Also, in order to announce disabled state uniformly, make the change to add disabled state in the renderer and make enabled state the implicit default.

BUG= 625411 , 630570 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2151713002
Cr-Commit-Position: refs/heads/master@{#407949}

[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/chrome/browser/resources/chromeos/chromevox/common/chrome_extension_externs.js
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_predicate.js
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/chrome/common/extensions/api/automation.idl
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/chrome/test/data/extensions/api_test/automation/tests/tabs/find.js
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/chrome/test/data/extensions/api_test/automation/tests/tabs/sanity_check.js
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/components/pdf/renderer/pdf_accessibility_tree.cc
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/content/browser/accessibility/browser_accessibility_android.cc
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/content/browser/accessibility/browser_accessibility_auralinux.cc
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/content/browser/accessibility/browser_accessibility_cocoa.mm
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/content/browser/accessibility/browser_accessibility_manager_win.cc
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/content/browser/accessibility/browser_accessibility_win.cc
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/content/browser/accessibility/browser_accessibility_win_unittest.cc
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/content/renderer/accessibility/blink_ax_enum_conversion.cc
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/ui/accessibility/ax_enums.idl
[modify] https://crrev.com/545a777dfe3e6410be16517f78f24602a64a55b6/ui/accessibility/platform/ax_platform_node_auralinux.cc
►

Sign in to add a comment