New issue
Advanced search Search tips

Issue 705343 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 3
Type: Task

Blocking:
issue 657462



Sign in to add a comment

Limit positions/selections to the inside of BODY during running execCommand

Project Member Reported by tkent@chromium.org, Mar 27 2017

Issue description

* If the current selection exceeds the BODY at the beginning of a command, we should shrink it to (BODY, 0) - (BODY, BODY.childCount).

* If the result of any functions to move positions/selections is not in the BODY, we should move/shrink it to the inside of the BODY.
  e.g. startOfParagraph() for a position in the BODY can return a position in HEAD now.

In https://codereview.chromium.org/2753193002/, we tried to change VisibleUnits.cpp so that a position didn't move across the BODY. However, we concluded that VisibleUnits should be simplified, and special handling for BODY should be done in the execCommands layer.

 
Components: Blink>Editing
Project Member

Comment 2 by sheriffbot@chromium.org, Dec 4

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Hotlist-Recharge-Cold
Status: Available (was: Untriaged)

Sign in to add a comment