/streamr

Cancel dragenter #1

Merged
merged 1 commit into from Apr 30, 2012

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@miketaylr @wookiehangover
@miketaylr
Contributor

miketaylr commented Apr 30, 2012

Per spec, you have to cancel dragenter as well as dragover if you don't use the dropzone attribute (which nobody but Opera 12 supports right now.

http://www.whatwg.org/specs/web-apps/current-work/multipage/dnd.html#dnd

Note: Instead of using the dropzone attribute, a drop target can handle the dragenter event (to report whether or not the drop target is to accept the drop) and the dragover event (to specify what feedback is to be shown to the user).

wookiehangover pushed a commit that referenced this pull request Apr 30, 2012

Sam Breed
Merge pull request #1 from miketaylr/master 
Cancel dragenter
d425139

@wookiehangover wookiehangover merged commit d425139 into wookiehangover:master Apr 30, 2012

@wookiehangover

This comment has been minimized.

Show comment Hide comment
@wookiehangover

wookiehangover Apr 30, 2012

Owner

huh, didn't know about that one. Thanks for the patch, Mike!

Owner

wookiehangover commented Apr 30, 2012

huh, didn't know about that one. Thanks for the patch, Mike!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment