New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 700214 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Last visit > 30 days ago
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 3
Type: Bug
Team-Accessibility



Sign in to add a comment

IA2 SetCaretOffset places the caret on the embedded object if you try to set it to the character after it

Project Member Reported by nek...@chromium.org, Mar 10 2017

Issue description

<p>he<img alt="l">lo</p>
p.IAccessibleText.SetCaretOffset(4) succeeds but p.IAccessibleText.get_caretOffset then returns 3.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 17 2017

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

commit 4089977fdebad10fac160d2a43d65b97f00d92ab
Author: nektar <nektar@chromium.org>
Date: Fri Mar 17 02:49:39 2017

WIP: Modified AXPosition to work with objects with both embedded object characters and text.

On Windows, I had assumed that we would use tree positions for non text nobjects and text positions for text objects. This can work only if IA2 hypertext contains either a series of embedded object characters or text, but not both.
However, the fact that we append the text of each static text object to its immediate parent for performance reasons, means that we could potentially have IA2 hypertext that contains both embedded object characters and text.
BUG=700214
R=dmazzoni@chromium.org
TESTED=not yet

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

[modify] https://crrev.com/4089977fdebad10fac160d2a43d65b97f00d92ab/content/browser/accessibility/ax_platform_position.cc
[modify] https://crrev.com/4089977fdebad10fac160d2a43d65b97f00d92ab/content/browser/accessibility/ax_platform_position.h
[modify] https://crrev.com/4089977fdebad10fac160d2a43d65b97f00d92ab/ui/accessibility/ax_node.cc
[modify] https://crrev.com/4089977fdebad10fac160d2a43d65b97f00d92ab/ui/accessibility/ax_node.h
[modify] https://crrev.com/4089977fdebad10fac160d2a43d65b97f00d92ab/ui/accessibility/ax_position.h

Labels: NewComponent-Accessibility NewComponent-Accessibility-Compatibility
Components: UI>Accessibility>Compatibility
Components: -UI>Accessibility
Labels: -newcomponent-accessibility-compatibility -newcomponent-accessibility
Labels: triage-lpalmaro
Labels: -triage-lpalmaro
Nektarios, is the status of started still correct here? 

Comment 7 by nek...@chromium.org, Dec 10 2017

Labels: win-a11y
Status: Assigned (was: Started)
Changing priority since this is concerning text navigation.

Comment 8 by nek...@chromium.org, Dec 12 2017

Status: Available (was: Assigned)

Comment 9 by nek...@chromium.org, Dec 15 2017

Labels: editing
Status: Assigned (was: Available)

Sign in to add a comment