New issue
Advanced search Search tips

Issue 624586 link

Starred by 4 users

Issue metadata

Status: Verified
Owner: ----
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug
Team-Accessibility



Sign in to add a comment

About page fails to focus properly with CVox running

Reported by nimerjaber1@gmail.com, Jun 29 2016

Issue description

Mode: force_next
Version: 53.0.2773.0
Reproduction Steps: 
1. Enable Cvox next
2. Navigate to the about page. Alt+F>>Help>>About
3. Attempt to read the about page.
Expected: Reading and navigational commands can be used to access the content of this page.
Actual: It seems that part of the page is focused, however one must press alt+tab in order to place true focus on the page in order to navigate and read the content.
 
Actually, this happens when accessing settings page from the omnibar as well.
Cc: lpalmaro@chromium.org dtseng@chromium.org
Status: Untriaged (was: Unconfirmed)
Checked this on 53.0.2782.6.

If we navigate to Help>about page, the focus doesn't stay on the about page but it goes to the previous page on which the focus was.
Labels: Phase3
Status: Available (was: Untriaged)
This is still an issue in 55.0.2878.0
Project Member

Comment 5 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 6 by dtseng@chromium.org, Oct 12 2016

Status: Fixed (was: Available)
Status: Verified (was: Fixed)
verified on 56.0.2903.0

Sign in to add a comment