-
Notifications
You must be signed in to change notification settings - Fork 385
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bounce Area Background / Reduced required pull distance #9
base: master
Are you sure you want to change the base?
Conversation
- Bounce area background - Requires shooter pull quester (landscape compatible)
Regarding the background issue, I don't understand why you don't simply change the refreshControl background color. For the second issue, it's a bit more complicated than that. |
(gentle ping) - do you have any additional thoughts on how to resolve this yet? |
Thoughts have been given, but time has not been found :/ |
This thread looks old, but if folks are still interested in this issue of being able to pull the refreshcontrol when in landscape I've implemented a solution that is working for me. I've applied an acceleration factor to the stretching of the control, so it effectively snaps and pops faster. I'm applying this universally but it would be better to apply different factors in landscape than in portrait (an acceleration factor of 1 will result in existing behavior). The code waits until the pull has exceeded a threshold amount of kOpenedViewHeight, after that it applies an effectiveOffset that uses the acceleration factor to speed up the stretch growth of the control. I'm not very good with git so I don't know how to offer this via my own commit so I'm pasting in code into this comment. // define the acceleration factor // in observeValueForKeyPath replace code that starts at the line: _lastOffset = offset; with:
|
The background for the bounce area is shown both in Apple's mail and contacts app.
If a project uses both a Navigation & toolbar, the scroll distance of 53 in landscape doesn't work. I've reduced it to 36.