Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

July 29th F2F Agenda Item - Dragging and dropping #134

Closed
gked opened this issue Jul 29, 2016 · 5 comments
Closed

July 29th F2F Agenda Item - Dragging and dropping #134

gked opened this issue Jul 29, 2016 · 5 comments

Comments

@gked
Copy link

gked commented Jul 29, 2016

What should the order of the events be and in what spec should this be placed?

@gked
Copy link
Author

gked commented Jul 29, 2016

Resolution: In what order do dragend, input and drop events are fired and when is dom updated during those events in Edge. Then we can come to real resolution.

@garykac
Copy link
Member

garykac commented Aug 1, 2016

Current order for Cr, FF, Safari is:

  • drop
  • (beforeinput should go here)
  • update DOM
  • input
  • dragend

Need to verify with Edge and confirm.

@garykac
Copy link
Member

garykac commented Aug 1, 2016

Spec also says something about updating the DOM on dragend if dragging selected text. Need to investigate and see what the browsers currently do. dragend may need to generate beforeinput/input events as well.

@gked
Copy link
Author

gked commented Aug 2, 2016

Edge has bugs in this area. Event sequence it fires is different from the rest:

  • drop
  • dragend
  • partial DOM update ( img element is being inserted after foo but not being removed before foo)
  • dragend
  • input
    -DOM update

We'll fix the issue on our side as the behavior of the rest of the browsers makes sense.

To Gary's second comment, Edge does update the DOM after dragend is being fired but for images. This is probably why Edge tries to update the DOM once dragend is fired.

@johanneswilm
Copy link
Contributor

This issue was moved to w3c/input-events#24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants