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 doesn't work in Microsoft Edge #1036

Closed
lilymatcha opened this issue Dec 31, 2018 · 6 comments
Closed

Touch doesn't work in Microsoft Edge #1036

lilymatcha opened this issue Dec 31, 2018 · 6 comments

Comments

@lilymatcha
Copy link

lilymatcha commented Dec 31, 2018

Expected behavior

You long-press to select in order to initiate the drag.

Actual behavior

When you long-press, it simply selects and does not drag.

Steps to reproduce

Open Microsoft Edge, go to any of the examples, and attempt to use touch to drag.

What version of React are you using?

16.6.3

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

10.0.3

What browser are you using?

Microsoft Edge (v. HTML 17.17134)

Demo

On Youtube.

@Weffe
Copy link

Weffe commented Jan 2, 2019

What device is that demo running on? Microsoft Surface Pro?

@lilymatcha
Copy link
Author

@Weffe Microsoft Surface Laptop

@treshugart
Copy link

Alex will be back mid-Jan. See #1010 for more details.

@alexreardon
Copy link
Collaborator

I tested this and it seemed to work for me.. Strange. Should continue to investigate!

I will try to get access to appropriate hardware

@lilymatcha
Copy link
Author

@alexreardon What hardware are you testing on? I think it shouldn't be a hardware (i.e. lack of drivers, etc.) issue, because the library works fine for me on Chrome, just not on Edge. Furthermore, adding the pointer sensor (#1045) fixed the issue for me, so I'm curious what's different that could be allowing it to work on your end.

@alexreardon
Copy link
Collaborator

I mentioned this in #1045:

Edge is moving to Chromium and there are already beta Edge builds. My expectation is that Edge moving to Chromium will fix #1036. Please let me know if that is not the case and I am more than happy to revisit this.
I want to stress how thankful I am @lilymatcha for your input. Let me know if there are other parts of this library you are keen to get involved in

Side thought about edge

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

4 participants