New issue
Advanced search Search tips

Issue 656349 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

edittext jump-command > type doesn't respond

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

Issue description

Mode: force_next
Version: 56.0.2890.0
Reproduction Steps: 
1. search+a+i
2. Tab > search+shift+e
3. Type

Observed: Nothing
Expected: text to be inputted into edit text

Regression: Yes
NOte: This use to work in M55 and earlier
 
Project Member

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

Labels: Hotlist-Google
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 24 2016

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

commit 2ff29563c5b0619e3722c74af2fe3afa18819999
Author: dtseng <dtseng@chromium.org>
Date: Mon Oct 24 18:48:56 2016

Fix editable text selection

We need to special-case setting selections when the focused node is itself editable.
- make no adjustments for the selected node; placing selection outside of the editable content causes key presses to go to the page
- if we are trying to select the entire node, set the selection index to 0 (placing selection at the beginning of the node); otherwise, use the character offset

TEST=move to a text field or content editable via ChromeVox commands; verify typing works to echo the keys. Furthermore, verify that in a content editable with headings, links, etc, jump commands move the selection. Finally, verify ChromeVox text movement commands (e.g. next character), moves selection in single line editables.
BUG= 656349 , 658045 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/2ff29563c5b0619e3722c74af2fe3afa18819999/chrome/browser/resources/chromeos/chromevox/cvox2/background/cursors.js

Comment 3 by dtseng@chromium.org, Oct 24 2016

Status: fixed (was: Untriaged)
Status: Verified (was: Fixed)
verified on 56.0.2903.0

Sign in to add a comment