Support for touch events?

[DSheffield]

Is there any plan to support touch events as well? So click outside could effectively be click or touch outside?

[zeppelin]

@DSheffield It does work on mobile devices - if that's what you're asking. Is there a specific reason you need touch events?

[DSheffield]

I see from your simple demo app it does seem to work with touch (or at least the corresponding mouse events the browser sends when touches occur). We've used the addon in our app and it seemed it wasn't invoking the clickOutside action in response to touch events. But, maybe we've done something that's interfering with the events. I'll check into that a bit further.

We need touch events cause our app is kinda positioned as touch based - but we're using touch screen laptops (not phones / ipads, etc).

[DSheffield]

So, here's an example of what we're looking to support. A simple touch / release triggers the action. A touch / swipe, however, does not:

But a similar drag with the mouse does trigger the action:

[zeppelin]

So you need the same click behavior on touch-based devices?

[DSheffield]

Yeah, effectively I want the clickOutside action to be triggered any time there's any kind of touch interaction not within my component.

[zeppelin]

To be honest I did not have such goals :) Nevertheless, if you have working fork supporting this -and it does fit into the original addon's API- I'm happy to pull the changes to upstream!

[DSheffield]

What is your browser / version support goal?

[zeppelin]

It's not quite about browser support per se. The way I think about this is more of how mobile browsers are handling these interactions with the current version. They do support clicks on elements, so the current behavior is not buggy. That said, if there was a setting, like <ClickOutside @useTouchEvents={{true}} ...>, it'd be a different story. I haven't planned adding these sort of stuff to the feature set myself. But, like I mentioned, I'm open to such contributions - it's only that it shouldn't break existing behavior. If that's the case, I'm cool with it!

[zeppelin]

@DSheffield Can be achieved with #51