New issue
Advanced search Search tips

Issue 833618 link

Starred by 3 users

Issue metadata

Status: Assigned
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Impossible to check ctrl/alt/shift keys state on `drop`, `dragend` and possibly other events

Reported by josiahds...@gmail.com, Apr 16 2018

Issue description

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

Steps to reproduce the problem:
1. Try simple JS snippet from http://jsfiddle.net/mp0e599g/
2. Drag one element over another by pressing and releasing Control/Shift/Alt key
3. Inspect JS console output

What is the expected behavior?
event.ctrlKey, event.shiftKey, event.altKey should represent actual state of corresponding keys.

What went wrong?
This properties always return false.

Did this work before? No 

Does this work in other browsers? Yes

Chrome version: 65.0.3325.181  Channel: stable
OS Version: 10.0
Flash Version: 

THis bug is mostly copied from https://bugs.chromium.org/p/chromium/issues/detail?id=413839#c22
It references a similar issue but pertains to the element being dropped on, not the actual element being drag/dropped. The examples are verifiably different with one working as expected and this one _not_ working as expected.
 
Labels: Needs-Triage-M65
Same problem using Chrome Version 65.0.3325.181 on Mac. 
https://stackoverflow.com/q/49852407/2256325
Components: Blink>Input
Labels: M-68 Triaged-ET FoundIn-68 Target-68 OS-Linux OS-Mac
Status: Untriaged (was: Unconfirmed)
Able to reproduce the issue on Mac 10.13.3, Win-10 and Ubuntu 14.04 using chrome reported version #65.0.3325.181 and latest canary #68.0.3397.0.
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...!!
Labels: Hotlist-Input-Dev
Owner: eirage@chromium.org
Status: Assigned (was: Untriaged)
eirage@ it seems that we do set these properties here in this code:
https://cs.chromium.org/chromium/src/third_party/blink/renderer/core/input/mouse_event_manager.cc?sq=package:chromium&dr=C&l=1064

Can you investigate why maybe the original event doesn't have those modifiers?

Comment 5 by eirage@chromium.org, Apr 19 2018

'dragend' original event doesn't set the modifier for all platform.
'drop' modifier is set on mac, win and linux. 
'dragstart' event modifiers is copy from mouse down event that initiate drag.
other events 'dragover' 'dragenter' etc should have correct modifiers on aura and mac.
So only 'dragend' event has incorrect 

Sign in to add a comment