search

bduncavage / PullToRefresharp

A pull-to-refresh library for Android in C#
38 stars 78 forks source link

Long touches not registering when Listview is at top scrollposition #8

Open leonluc-dev opened 10 years ago

leonluc-dev commented 10 years ago

I'm using a ListView and ExpandableListView with pull to refresh enabled (using a ViewWrapper and all).

When the ListViews are at their top scroll position (Y for the scroll component is 0) the ListView no longer registers long touches (like the ones registered by IOnLongItemClickListener). At any other scroll position the long touches work fine.

I figured out that the issue is caused by the OnTouchEvent and OnInterceptTouchEvent in the ViewWrapper class. More specifically in the line 233-241 and line 281-288 blocks in the ViewWrapper.cs file. But I can't figure out how the bug occurs.

I tried to fix it by changing the return ContentView.IsAtTop; on line 288 to return false; While this does allow the long touches to fire their relevant events, it will also allow long touches to fire events when scrolling to "pull down" the refresh header. So in the end it doesn't work out.

bduncavage commented 10 years ago

Thanks for the report. I'll look into a fix soon.

leonluc-dev commented 10 years ago

Are there any updates on this issue as of yet?

rusty21 commented 9 years ago

This is still an issue, will this ever be patched??