Skip to content

Conversation

@rocman
Copy link
Contributor

@rocman rocman commented May 23, 2016

This makes it possible to do the interpolation on properties of any type. For example, the contentOffset of a ScrollView.

@ghost
Copy link

ghost commented May 23, 2016

By analyzing the blame information on this pull request, we identified @lelandrichardson and @vjeux to be potential reviewers.

@ghost ghost added GH Review: review-needed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels May 23, 2016
@ghost
Copy link

ghost commented May 23, 2016

@rocman updated the pull request.

@rocman
Copy link
Contributor Author

rocman commented May 25, 2016

@fatihict big thanks for your reviewing. The change you mentioned above is a mistake. And I've fixed it just now. Thanks.

@ghost
Copy link

ghost commented May 25, 2016

@rocman updated the pull request.

@lelandrichardson
Copy link
Contributor

An API like this won't be compatible with native animations, unfortunately. I wonder if there is any way we could make it so that it would be?

Have to run right now, but will look at this PR a bit more and think about it. You may want to look at this PR for comparison, which could achieve something similar: animatedjs/animated#12

@ghost
Copy link

ghost commented Jun 24, 2016

It's been a while since the last commit was reviewed and the labels show this pull request needs review. Based on the blame information for the files in this pull request we identified @lelandrichardson as a potential reviewer. Could you take a look please or cc someone with more context?

@ghost ghost 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 Jul 12, 2016
@mkonicek
Copy link
Contributor

mkonicek commented Sep 9, 2016

@rocman I'm going through all pull requests and noticed this one hasn't been updated in a while and the last comment mentions:

An API like this won't be compatible with native animations, unfortunately. I wonder if there is any way we could make it so that it would be?

I'll close this pull request so it doesn't stay open indefinitely but please send a new one if you want to continue working on this.

@mkonicek mkonicek closed this Sep 9, 2016
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