New issue
Advanced search Search tips

Issue 656353 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

Tab into edit text should utter <text> selected>

Project Member Reported by chaok@google.com, Oct 15 2016

Issue description

Mode: force_next
Version: 56.0.2890.0
Reproduction Steps: 
1. Type into an edit text
2. tab > shift+tab
3. The text is highlighted, but cvox2 user isn't notified of this


 
Project Member

Comment 1 by sheriffbot@chromium.org, Oct 16 2016

Labels: Hotlist-Google
Labels: Phase3
Owner: dtseng@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Oct 26 2016

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

commit 3e79fe872b6b9b759a4962cc820d32ee97bd71af
Author: dtseng <dtseng@chromium.org>
Date: Wed Oct 26 18:15:48 2016

Read the selected text explicitly when focusing a texdt field

This makes value output prefer output of selected text if available; otherwise, read the entire value.
TEST=focus text fields in a variety of ways:
- tab
- ChromeVox commands
Verify that on first focus, we don't manipulate the selection (chromevox_tests cover this well).
Verify that in all other cases, including continuing to move inside of an editable, selection is moved.

BUG= 656353 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/3e79fe872b6b9b759a4962cc820d32ee97bd71af/chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_util.js
[modify] https://crrev.com/3e79fe872b6b9b759a4962cc820d32ee97bd71af/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
[modify] https://crrev.com/3e79fe872b6b9b759a4962cc820d32ee97bd71af/chrome/browser/resources/chromeos/chromevox/cvox2/background/output.js

Comment 4 by dtseng@chromium.org, Oct 26 2016

Status: fixed (was: Assigned)

Comment 5 by dchan@google.com, Jan 21 2017

Labels: VerifyIn-57

Comment 6 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58
Status: Verified (was: Fixed)

Sign in to add a comment