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

Doesn't work with touch events #286

Open
ScottBeeson opened this issue Nov 28, 2017 · 5 comments
Open

Doesn't work with touch events #286

ScottBeeson opened this issue Nov 28, 2017 · 5 comments
Labels
feature New feature or enhancement help wanted

Comments

@ScottBeeson
Copy link

ScottBeeson commented Nov 28, 2017

Works great on my UI in Chrome until I click the image button ("toggle device toolbar"). After that, nothing.

@lukasoppermann lukasoppermann added feature New feature or enhancement help wanted labels Nov 28, 2017
@lukasoppermann
Copy link
Owner

Hey, sadly it is a bit more complex, as drag events don't work on mobile same for this example: https://developer.mozilla.org/en-US/docs/Web/Events/dragstart.

To add this feature I think you would need to build something using touch events.

I would love it if you could send a PR for it. I will gladly assist you if you need any input.

@mike16889
Copy link
Contributor

mike16889 commented Jan 22, 2021

@ScottBeeson & @lukasoppermann I know this issue is old, but I came across a "Drag & Drop touch polyfill" on github https://github.com/Bernardo-Castilho/dragdroptouch/blob/master/DragDropTouch.js and it just seems to work.

Could be a solution to just include it or look at what its doing and replicate it.

@lukasoppermann
Copy link
Owner

Hey @mike16889 thank you for the link. Would you like to send a PR to add this?

@mike16889
Copy link
Contributor

@lukasoppermann I'm not really sure what would be required to integrate it, it may just be a better idea to put a note in the reader, something along the lines of touch support can be added by including the drag & drop touch polyfill" with a link to the git.

@lukasoppermann
Copy link
Owner

Hey @mike16889, great idea. Would you like to send a PR updating the Readme.md file? If so, please link to the repos main page https://github.com/Bernardo-Castilho/dragdroptouch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or enhancement help wanted
Projects
current Version
  
Backlog
Development

No branches or pull requests

3 participants