New issue
Advanced search Search tips

Issue 810946 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: 2018-02-28
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

NEEDS_FEEDBACK Selection Range setStart and setEnd are inconsistent for forward and backward selection

Reported by hatp...@gmail.com, Feb 9 2018

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3239.132 Safari/537.36

Steps to reproduce the problem:
1. Navigate to https://codesandbox.io/s/yvnknqmrl1
2. Make sure the checkbox shows LTR
3. Try to select text from left to right
4. Now try to select text from right to left.

What is the expected behavior?
Text to be selected on a per token base.

What went wrong?
Backward selection doesn't select the text.

Did this work before? N/A 

Chrome version: 63.0.3239.132  Channel: n/a
OS Version: 10.0
Flash Version: Shockwave Flash 28.0 r0

**This code has the logic to make sure the user is seeing a per token selection while selecting the text.
Change the direction to RTL and observe the same behavior. (maybe even worse)
 
React App - Google Chrome 2_9_2018 3_00_52 PM.gif
1.6 MB View Download
Labels: Needs-Triage-M63

Comment 2 by rtoy@chromium.org, Feb 13 2018

Components: -Blink Blink>Editing>Selection
Status: Untriaged (was: Unconfirmed)

Comment 3 by yosin@chromium.org, Feb 14 2018

Labels: Needs-Feedback
NextAction: 2018-02-28
Status: Unconfirmed (was: Untriaged)
Summary: NEEDS_FEEDBACK Selection Range setStart and setEnd are inconsistent for forward and backward selection (was: Selection Range setStart and setEnd are inconsistent for forward and backward selection)
Could you provide simple HTML to see behavior?
The URI specified in #c1 keeps showing "Bundling Dependencies...".

Comment 4 by hatp...@gmail.com, Feb 14 2018

Sorry, I can't find the time to re-do that, I've made this React app solely for the purpose of this bug report. I've attached the code here if that works.
yvnknqmrl1.zip
13.0 KB Download
Project Member

Comment 5 by sheriffbot@chromium.org, Feb 14 2018

Cc: yosin@chromium.org
Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "yosin@chromium.org" to the cc list and removing "Needs-Feedback" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 6 by yosin@chromium.org, Feb 15 2018

Labels: -Needs-Triage-M63 Needs-Feedback
>#c4, thanks for attching ZIP file.
But, it doesn't work.
"index.html" doesn't have SCRIPT elements to load script files.

Comment 7 by hatp...@gmail.com, Feb 15 2018

Try this:
https://yvnknqmrl1.codesandbox.io/
My Chrome Version 64.0.3282.140 (Official Build) (64-bit) shows the content
Project Member

Comment 8 by sheriffbot@chromium.org, Feb 15 2018

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "yosin@chromium.org" to the cc list and removing "Needs-Feedback" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 9 by yosin@chromium.org, Feb 15 2018

Cc: -yosin@chromium.org
Labels: Needs-Feedback
>#c7, thanks for URI. It works for me.
However, I'm not sure how this page works since it consists many JS files.

Please provide minmal reproduce case without React to identify the root cause.

BTW, when I attempt to run this app on Edge, it causes IndexSizeError, it may
cause unexpected behavior on Chrome.


Here is an error message on Edge:

Error
IndexSizeError

Anonymous function
https://codesandbox.io/static/js/sandbox.bea94ee3.js:5060:99

The NextAction date has arrived: 2018-02-28

Comment 11 by yosin@chromium.org, Mar 15 2018

Status: WontFix (was: Unconfirmed)
Mark WontFix since we could not get HTML+resources to reproduce.

Sign in to add a comment