Fixed the scrollview animation issue and added some helpers. #2
Conversation
This is to prevent the animation bug described at michaeltyson#1
|
Cool! I'll take a look at it soon Michael Tyson | atastypixel.com (https://atastypixel.com/) Loopy 2 (https://loopyapp.com): A saavy, sophisticated, tactile new looper that reinvents the formula. Make music on your iPhone from anything. Coming soon. Subscribe to our newsletter (https://atastypixel.com/newsletter) e: [email protected] (mailto:[email protected]) aim: mikerusselltyson On Monday, 6 June 2011 at 14:50, webartisan wrote:
|
-removed the conditions that check for original content inset before doing anything. These made since when the height was being adjusted, but not anymore. They were just causing problems.
-I don't know if it was originally like this but I'm going to fix it anyway.
|
I fixed a couple of issues that I found. The contentInsets weren't being reset when the keyboard was hiding. Also the scrollview should now support fields that are inside a subview of the scrollview rather than just direct child fields. Also my 3rd commit has a confusing comment, I meant to say that I changed it back to pin-stripe. I think in my last push I changed it to white and it looked kind of ugly. Not all that important though. |
Fixed the scrollview animation issue and added some helpers.
|
Thanks heaps for this! Sorry it took me so long to get my act together and merge it in =) |
I found that the animation issue could be fixed if you increase the contentInset.bottom instead of shrinking the height.
I also added a couple extra things that I've found helpful.