New issue
Advanced search Search tips

Issue 640587 link

Starred by 5 users

Issue metadata

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



Sign in to add a comment

When reaching an edit field using CV navigation, it is not editeable

Reported by henk.a...@gmail.com, Aug 24 2016

Issue description

1. Open a page that contains edit fileds or editeable areas, such as this bug entry page.
2. Move using cv+arrow, or cv+e to an edit field 
3. Try to type text. 
4. Result you hear nothing and no text is entered.
5. Press tab/shift tab to get to the field and notice the field is editeable.
6. Expected result: after reaching an edit field using CV navigation you should be able to type text immediately,
I think this worked fine in previous versions.Mode: force_next
Version: 54.0.2830.0
Reproduction Steps: 
1.
2.
3.


 
Labels: -Pri-2 Phase3 Pri-1
Owner: dtseng@chromium.org
Status: Assigned (was: Unconfirmed)
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 27 2016

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

commit 2063c5ec1c11f5b9823d8df8cbe647c5e9168c63
Author: dtseng <dtseng@chromium.org>
Date: Sat Aug 27 00:10:10 2016

Ensure focus() gets called when navigating to text fields

This was a recent regression. Logic was added to make a top level editable a container (so that navigation would descend into it). However, we were previously using the container predicate to decide whether or not to set focus; this prevents focus() on roots which sometimes triggered other side effects such as focusing a descendant.

Make the focus precondition check the structuralContainer predicate instead.

BUG= 640587 
TEST=chromevox_tests

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

[modify] https://crrev.com/2063c5ec1c11f5b9823d8df8cbe647c5e9168c63/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
[modify] https://crrev.com/2063c5ec1c11f5b9823d8df8cbe647c5e9168c63/chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs

Cc: dtseng@chromium.org
 Issue 628905  has been merged into this issue.
 Issue 641850  has been merged into this issue.
Status: Fixed (was: Assigned)

Comment 6 by chaok@google.com, Sep 12 2016

Keyboard focus now follows quick-nav focus in: 55.0.2857.0

Thanks!

Comment 7 by dtseng@chromium.org, Sep 12 2016

Status: verified (was: Fixed)

Sign in to add a comment