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

Closed
nestserau opened this Issue Nov 2, 2012 · 4 comments

Comments

Projects
None yet
3 participants

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 Nov 2, 2012

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

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.

Owner

Sephiroth87 commented Nov 4, 2012

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

Thank you very much man! You got it right.

ipodishima added a commit to ipodishima/ODRefreshControl that referenced this issue Nov 5, 2012

Vortec4800 pushed a commit to Vortec4800/ODRefreshControl that referenced this issue Nov 13, 2012

Merge branch 'master' of git://github.com/Sephiroth87/ODRefreshControl
# By Fabio Ritrovato (10) and others
# Via Fabio Ritrovato
* 'master' of git://github.com/Sephiroth87/ODRefreshControl:
  Only ignore "Fast scrolling" when the offset is positive (fix #41 + fix #42)
  Nope
  New README image url
  Don't trigger when scrolling up too fast, should fix #33
  Update README with new property
  Fix activityIndicatorViewColor for iOS4
  property activityIndicatorViewColor added
  Update podspec
  Higher view fill iPad bigger scroll area
  Reverting some things
  It's probably better to use the class to check
  UIScrollView does not implment numberOfSections
  added custom activity indicator support

Conflicts:
	ODRefreshControl/ODRefreshControl.m
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment