Skip to content

Comments

[RFC][ListView] Add getInnerViewNode, scrollTo, scrollWithoutAnimationTo#765

Closed
ide wants to merge 1 commit intofacebook:masterfrom
ide:listview-api-1
Closed

[RFC][ListView] Add getInnerViewNode, scrollTo, scrollWithoutAnimationTo#765
ide wants to merge 1 commit intofacebook:masterfrom
ide:listview-api-1

Conversation

@ide
Copy link
Contributor

@ide ide commented Apr 9, 2015

After building some scroll view components (pull-to-refresh, infinite scroll) one of the most important features I found was composition of different scroll views. This diff adds scroll view methods to ListView so that it too acts like a scroll view and can be composed.

This is an RFC because I have another approach that is a little more robust but perhaps more obtuse. See #766.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 9, 2015
@nicklockwood
Copy link
Contributor

I think I need a little more context about how this is used. Do you have an example implementation you can point me to?

@ide ide force-pushed the listview-api-1 branch 4 times, most recently from d8149ce to 4f542cd Compare April 15, 2015 21:00
@ide ide force-pushed the listview-api-1 branch 2 times, most recently from 651667d to 010f811 Compare April 28, 2015 01:24
@ide ide force-pushed the listview-api-1 branch from 010f811 to 9823ce8 Compare April 30, 2015 05:36
After building some scroll view components (pull-to-refresh, infinite scroll) one of the most important features I found was composition of different scroll views. This diff adds scroll view methods to ListView so that it too acts like a scroll view and can be composed.

This is an RFC because I have another approach that is a little more robust but perhaps more obtuse.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants