New issue
Advanced search Search tips

Issue 652143 link

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



Sign in to add a comment

CVox panel reads out content of last focused item

Reported by ni...@jaberspace.com, Oct 3 2016 
Mode: force_next
Version: 55.0.2878.0
Reproduction Steps: 
1. Enable CVox Next.
2. google.com>>search for "coke"
3. Search+. to focus CVox panel.
4. Right arrow to headings and down arrow to my coke rewards.
5. Press enter to move focus to that heading.
6. Move to another position on the page and invoke the panel again.
Expected: Chromevox panel is announced.
Actual: Focus is placed on the panel with the list of CVox commands, however my coke rewards is announced to the user first.
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 12 2016 

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

commit 7a99c7d3e7dc2ca71bbaa127099239355a41fd37
Author: dtseng <dtseng@chromium.org>
Date: Wed Oct 12 00:20:23 2016

Recovery: Implement focus recovery across root AutomationNodes

This cl maintains a WeakMap of AutomationRootNode objects to the last known node that had ChromeVox range.

This map gets updated manually when marked or implicitly when focus crosses top level root nodes.

TEST=manual; open context menu, open panel, activate menus that manipulate focus, press escape; ctrl+tab, alt+tab. Verify focus lands as expected.
BUG= 631918 , 628912 , 624586 , 605377 , 524673 , 652143 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/7a99c7d3e7dc2ca71bbaa127099239355a41fd37/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
[modify] https://crrev.com/7a99c7d3e7dc2ca71bbaa127099239355a41fd37/chrome/browser/resources/chromeos/chromevox/cvox2/background/command_handler.js
[modify] https://crrev.com/7a99c7d3e7dc2ca71bbaa127099239355a41fd37/chrome/browser/resources/chromeos/chromevox/cvox2/background/desktop_automation_handler.js
[modify] https://crrev.com/7a99c7d3e7dc2ca71bbaa127099239355a41fd37/chrome/browser/resources/chromeos/chromevox/cvox2/background/i_search.js
[modify] https://crrev.com/7a99c7d3e7dc2ca71bbaa127099239355a41fd37/chrome/browser/resources/chromeos/chromevox/cvox2/background/panel.js

Comment 2 by dtseng@chromium.org, Oct 12 2016

Status: Fixed (was: Unconfirmed)

Comment 3 by chaok@google.com, Oct 15 2016

Status: Verified (was: Fixed)
Fixed in: 56.0.2890.0
►

Sign in to add a comment