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

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

Closed
wants to merge 1 commit into from

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
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.
@ide ide closed this May 15, 2015
@ide ide deleted the listview-api-1 branch May 15, 2015 04:51
ayushjainrksh referenced this pull request in MLH-Fellowship/react-native Jul 2, 2020
Following up on #749, I missed a section. This adds it there too.
mganandraj pushed a commit to mganandraj/react-native that referenced this pull request Oct 28, 2021
* Fix animation bug  (facebook#763)

* pod install

* fix pod errors

* fix pod errors

* fix pod errors
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