[TIMOB-16680] iOS: listView when views overlap itemclick event return wrong bindId

[yomybaby]

Fix this issue TIMOB-16680

FindViewProxyWithBindIdContainingPoint function have to find bindId from front to back(reverse index).

[ingo]

Thank you for the PR. We will review this shortly.

[Topener]

Is there an update? "Shortly" shouldn't be months I guess :)

[ingo]

It's scheduled for an upcoming release so we will get to it...apologies, other items have come up.

[cheekiatng]

Hey @yomybaby , thanks for the PR. While I'm reviewing, i tried the change you made but it doesn't fix TIMOB-16680. However, I noticed that in your code, you had a check for subview.userInteractionEnabled, may I know the reason for this? Because without this, it fixes it.

[yomybaby]

Hi @cheekiatng ,

About fix bug

This PR fix TIMOB-16680. Other guys said it on JIRA comment. I have not tested on SDK 4.x. What version you use?

About subview.userInteractionEnabled

if touchEnabled is false, the view doesn't react any touch action.
Look at below example. If you touch Item bindId should be 'viewA', even viewA is located behind viewB.

<ItemTemplate>
   <View bindId="viewA"></View>
   <View bindId="viewB" touchEnabled="false"></View>
</ItemTemplate>

[cheekiatng]

Hi @yomybaby I was using 4.0.0.Beta4 when I reproduced this. Anyway, I think i understand what you mean now, about the usage of touchEnabled. Ok will do one final check before the merge! :)

[cheekiatng]

put a (int) cast here, (int)[views.subviews count]-1
to remove Xcode warning.

[cheekiatng]

Hi @yomybaby , please address comments. Also, please include the year 2015 in Copyright (c) 2013-2015 by Appcelerator in the top part of the file. Then I will be able to merge. Thanks for your contribution!

[ashcoding]

Cherry-picked commit and addressed the comments: #6918

[ingo]

Closing as #6918 supplants this.