[TIMOB-10361] added touchPassThrough property on iOS which allows touches to go throug...

[farfromrefug]

...h when transparent

jira ticket

[negupta]

Signed CLA is in place.

[WhichKatieDid]

CR passed.

[WhichKatieDid]

Have been communicating with Android teams on how feasible this is on Android, will have to do same with MW.

[WhichKatieDid]

Okay, having talked with the other teams, this sounds highly unfeasible in terms of parity, i.e., making something there is either impossible or could cause significant performance problems. What's the underlying use case? The Jira ticket didn't really show a need.

[farfromrefug]

That doesnt seem hard to do an android
http://stackoverflow.com/questions/6374592/passing-touch-events-to-the-parent-view
I am going to go for it and test implementing it.

Now their are thousands of use cases:

• Buttons over camera view. You want them to respect a certain layout (horizontal for example), so you put them into a "group" view. Yet you dont want that group view to get any touch event (because touching the camera will focus).
• puller: You have a sliding view hidden on the left. You can show it with a puller. To do such thing you need an horizontal layout with the content of the view and the puller. But the puller is often not the same height as the view itself.
  Without passthrough, touching above and below the puller will get blocked

I actually use the 2 cases.

[farfromrefug]

I just commited a working android version.

[dawsontoth]

I can vouch for the validity of these use cases, and others. I've encountered these in apps I've built for Appcelerator, and having this property would greatly reduce the complexity of my code in some cases and make other cases, formerly impossible, possible.

[WhichKatieDid]

Okay, will talk with mobile web and see if this is possible or painful.

[farfromrefug]

any news on this?
Thanks

[WhichKatieDid]

Minor nitpick: This could be more elegantly written as
if (touchPassThrough && (hitview == self))
{
return nil;
}
return hitView;

[WhichKatieDid]

Despite Bryan making an odd face when I asked about this, this does have value. Include documentation and update the code (and update the merge) and I'll rereview.

[farfromrefug]

I am not the best at writing doc. Hope it will be fine!

[hansemannn]

@ashcoding Can you check the android part? I'd look at the iOS part.
@farfromrefug Can you resolve the merge conflicts so we get this merged in the 6.0.0 timeframe? Thank you!

[hansemannn]

Closing this PR, since the merge conflicts have not been addressed since 2 months.