Sephiroth87 / ODRefreshControl

A pull down to refresh control like the one in Apple's iOS6 Mail App
MIT License
2.15k stars 385 forks source link

When scrolling down from top, the control can suddenly disappear #42

Closed nestserau closed 11 years ago

nestserau commented 11 years ago

I played a lot with the control, and the issue I see is that when you scroll quickly but for a short distance from the top-most position, the refresh control disappears and it looks like a blink. iOS 6 refresh control doesn't behave like that. Instead, from what I can see, if you drag from the top-most position, if always updates. But only if you accelerate from the middle of the list, then it hides the control. Hope you understand what I'm talking about. :)

nfox513 commented 11 years ago

This bug was introduced in d39d6d7 . This bug makes it essentially impossible to initiate refreshing while in landscape on iPhone.

nestserau commented 11 years ago

I have a guess, that tracking `decelerating' state of the scroll-view instead of comparing new and old offset will do the trick. There is also a very good CKRefreshControl on github. Going to merge the ideas of the two projects into a single one.

Sephiroth87 commented 11 years ago

I'm not entirely sure I got the issue, but I think it's fixed now :)

nestserau commented 11 years ago

Thank you very much man! You got it right.