Issue metadata
Sign in to add a comment
|
Chromevox reading commands for moving to the next line do not work as expected in Inbox
Reported by
nimerjaber1@gmail.com,
May 20 2016
|
||||||||||||||||||||||||
Issue descriptionMode: force_next Version: 52.0.2739.0 Reproduction Steps: 1. Enable Chromevox next 2. Navigate to inbox.google.com and locate a message or thread you wish to read by pressing J and K. 3. Press enter to open the message. 4. Press Search+down arrow or Search+right arrow to navigate to the next line or item. Expected Result: The message could be read when moving by line. Actual Result: Focus is often placed back in the message list, the message that was expanded is now closed, and needs to be re-opened.
,
Jun 3 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/709ec11a4037d0fd97cff6a395d63ef574044d72 commit 709ec11a4037d0fd97cff6a395d63ef574044d72 Author: dtseng <dtseng@chromium.org> Date: Fri Jun 03 17:17:54 2016 Refine ChromeVox's treatment of containers 1. merge ignored containers and ignored leaf nodes into one predicate 2. only visit containers that are also visited by the provided predicate. For example, an element will match against a focusable toolbar, but a line won't. 3. scan the initial subtree of a node when moving to the next node if either of the following are true: the node is a container or the node doesn't match the predicate (e.g. element). This takes care of syncing when the current range is on an arbitrary position in the tree. When moving backwards, the leaf predicate restricts how deeply the backwards movement syncs downwards in the tree. BUG= 613473 CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2028173003 Cr-Commit-Position: refs/heads/master@{#397737} [modify] https://crrev.com/709ec11a4037d0fd97cff6a395d63ef574044d72/chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_predicate.js [modify] https://crrev.com/709ec11a4037d0fd97cff6a395d63ef574044d72/chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_util.js [modify] https://crrev.com/709ec11a4037d0fd97cff6a395d63ef574044d72/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js [modify] https://crrev.com/709ec11a4037d0fd97cff6a395d63ef574044d72/chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs [modify] https://crrev.com/709ec11a4037d0fd97cff6a395d63ef574044d72/chrome/browser/resources/chromeos/chromevox/cvox2/background/cursors.js [modify] https://crrev.com/709ec11a4037d0fd97cff6a395d63ef574044d72/chrome/browser/resources/chromeos/chromevox/cvox2/background/cursors_test.extjs
,
Jun 3 2016
,
Jun 20 2016
verified on 53.0.2768.0 |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by pbath...@chromium.org
, May 26 2016Status: Untriaged (was: Unconfirmed)