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

Issue 843556 link

Starred by 2 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 2
Type: Bug



Sign in to add a comment

target attribute of the dragenter event is not correct

Reported by cahitgur...@gmail.com, May 16 2018

Issue description

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

Steps to reproduce the problem:
1. Open the attached "a.html" file with using Chrome
2. Try to drag the text "Williams - 1" just a few px
3. Console is logging "Petra - 1" as an event target
4. Now try to drag the text "Williams - 2" just a few px
5. Console is logging "Thomas - 1" as an event target

What is the expected behavior?
I expect what the specification specifies.

https://www.w3.org/TR/html51/editing.html#eventdef-global-dragenter 
https://www.w3.org/TR/html51/editing.html#immediate-user-selection

In English: the target of the dragenter event must be the element underneath the element being dragged.

What went wrong?
How should I say... If the event target is wrong then everything goes wrong.

Did this work before? No 

Does this work in other browsers? Yes

Chrome version: 66.0.3359.139  Channel: n/a
OS Version: 10.0
Flash Version: 

If I remove the padding:0 from the ul element the dragenter event target is correct for the list case. But the issue still happes for the table rows.

Test page works fine on IE.
 
a.html
1.6 KB View Download
Labels: Needs-Triage-M66
Labels: Triaged-ET Needs-Feedback
Unable to reproduce the issue on Win-10 using chrome reported version #66.0.3359.139, latest stable #66.0.3359.181 and latest canary #68.0.3432.0.

Attached a screen cast for reference.

Following are the steps followed to reproduce the issue.
------------
1. Opened the attached "a.html" file with using Chrome
2. Dragged the text "Williams - 1" just a few px
3. Observed that "Williams - 1" is logged as an event target.
4. Now dragged the text "Williams - 2" just a few px
5. Observed that "Williams - 2" is logged as an event target.

cahitgurguc@ - Could you please check the attached screen cast and please let us know if anything missed from our end. Also please check the issue on latest canary #68.0.3432.0 by creating a new profile without any apps and extensions and please let us know if the issue still persist or not.

Thanks...!!
843556.mp4
663 KB View Download
Strange... I wish I could have the same result. Attaching a screen cast for a reference.
2018-05-17_15-59-14.mp4
701 KB View Download
Project Member

Comment 4 by sheriffbot@chromium.org, May 17 2018

Cc: krajshree@chromium.org
Labels: -Needs-Feedback
Thank you for providing more feedback. Adding the requester to the cc list.

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

Comment 5 Deleted

I could figure out how to reproduce the issue. Please change the "Scale and Layout" option from 100% to 150% on windows 10. (Desktop > right click > Display settings). It seems like the issue is happening only for the dragenter event. 

Labels: M-68 FoundIn-68 Target-68 OS-Linux
Status: Untriaged (was: Unconfirmed)
Able to reproduce the issue on Win-10 and Ubuntu 17.10 using chrome reported version #66.0.3359.139 and latest canary #68.0.3433.0. Issue is not seen in OS-Mac.
This is a non-regression issue as it is observed from M60 old builds. 

Hence, marking it as untriaged to get more inputs from dev team.

Thanks...!!

Comment 8 by fergal@chromium.org, May 21 2018

Cc: fergal@chromium.org
Status: Available (was: Untriaged)
Thanks. I'm going to leave this as a P2 as I'm not really sure of the severity. It seems to have been broken for quite a while without much trouble. Please respond if you think this needs more immediate attention.
Dear Daly, thanks for looking into this issue.

Drag-and-Drop capability is added into SAPUI5 framework, whether droppable  is allowed or not decision is taken at the dragenter event. Also preview option (drag an element to a new spot within the list, and the other items will adjust to fit) is using dragenter event as well.
 
So this might effect all SAP Fiori applications (according to following link 1886 applications) and their users. 
https://fioriappslibrary.hana.ondemand.com/sap/fix/externalViewer/#/home

I hope this explains the severity and immediate attention would be highly appreciated. 
Thanks...!!

Sign in to add a comment