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

Touch creates portal, but a second touch is required to re-initiate drag #2277

Open
acamann opened this issue Jul 7, 2021 · 3 comments
Open

Comments

@acamann
Copy link

acamann commented Jul 7, 2021

Thank you for the great library - I'm working on moving our product over from react-dnd so we can more consistently support touch

Expected behavior

intent touch creates portal and touch should be transferred to ref of object within portal to begin drag

Actual behavior

intent touch starts drag enough to create portal, but a second touch is necessary to now start dragging the portal

2021-07-07.10-57-41.mp4

Steps to reproduce

  1. https://react-beautiful-dnd.netlify.app/iframe.html?id=portals--using-your-own-portal&viewMode=story
  2. use chrome tools or mobile device and attempt drag using touch

Suggested solution?

not sure, but something seems to be getting lost on the reference to the object being dragged once it is moved into a portal

What version of React are you using?

16.12.0 (but also observed on above demo)

What version of react-beautiful-dnd are you running?

13.1.0

What browser are you using?

chrome, but observed on safari on iPad

@acamann
Copy link
Author

acamann commented Jul 7, 2021

The above is still an issue, but I got around it in my implementation by removing the portal and resolving other issues I had been seeing that made me try to use the portal (resulting from will-change: transform; style that I had previously applied to try to force safari to round corners, but was instead interfering with the expected default behavior of react-beautiful-dnd)

@azamuddin
Copy link

I have this problem too, it's really crucial bug for smooth mobile experience, tested on Safari IOS 16

@nflow
Copy link

nflow commented Apr 25, 2024

yea .. me too

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

No branches or pull requests

3 participants