Skip to content
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

Expose scrollsToTop on iOS. #2605

Closed
wants to merge 1 commit into from
Closed

Expose scrollsToTop on iOS. #2605

wants to merge 1 commit into from

Conversation

meeech
Copy link
Contributor

@meeech meeech commented Jul 19, 2012

@negupta
Copy link
Contributor

negupta commented Jul 28, 2012

@meeech - I do not see a signed CLA from this account. Did you sign one?

@meeech
Copy link
Contributor Author

meeech commented Jul 28, 2012

yes I def signed one a few months ago.

m

Sent from my iPhone

On 2012-07-28, at 6:48 PM, Neeraj Guptareply@reply.github.com wrote:

@meeech - I do not see a signed CLA from this account. Did you sign one?


Reply to this email directly or view it on GitHub:
#2605 (comment)

@meeech
Copy link
Contributor Author

meeech commented Jul 29, 2012

@negupta #1816 (comment)

@negupta
Copy link
Contributor

negupta commented Jul 29, 2012

Signed CLA is in place.

//http://developer.apple.com/library/ios/#documentation/uikit/reference/UIScrollView_Class/Reference/UIScrollView.html
-(void)setScrollsToTop_:(id)value
{
[[self tableView] setScrollsToTop:[TiUtils boolValue:value]];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To have this properly default to true, you'll need [TiUtils boolValue:value def:YES], otherwise doing tv.scrollsToTop=tv.scrollsToTop would set it false if undefined.

@WhichKatieDid
Copy link
Contributor

Closing this pull request and moving it to #2690.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants