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

Feature: interpolate: Extrapolate left/right #1549

Merged
merged 22 commits into from
Jan 13, 2021
Merged

Feature: interpolate: Extrapolate left/right #1549

merged 22 commits into from
Jan 13, 2021

Conversation

Roka20012
Copy link
Contributor

@Roka20012 Roka20012 commented Dec 19, 2020

Wix Goodness Squad Event

Motivation: to see extrapolating for both sides, just like in v1.

Fixes #1255

Changes

  • Added ts types for extrapolation
  • Added ExtrapolationExample
  • Added Error handling for invalid config or invalid interpolation value

Live example

IDK which example is better currently 2 in this PR

Screen.Recording.2020-12-19.at.16.23.55.mov

Screen.Recording.2020-12-19.at.18.48.46.mov

Checklist

  • Included code example that can be used to test this change
  • Updated TS types

@Roka20012
Copy link
Contributor Author

@terrysahaidak sup dude!
I think this PR need to review

Thanks again for being a maintainer on the Wix Goodness Squad Event

Cheers ✌️

@terrysahaidak
Copy link
Contributor

Hey, @karol-bisztyga this is yet another PR as a result of our event, could you please assign someone for review. Thanks.

Copy link
Member

@jakub-gonet jakub-gonet left a comment

Choose a reason for hiding this comment

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

Left some comments, but somebody from turboteam should review this.

src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
Example/src/App.js Outdated Show resolved Hide resolved
src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
@karol-bisztyga
Copy link
Contributor

It's a solid PR, I'll be happy to merge it, just please make those couple of thing more clear and I think we're good to go 😉

@Roka20012
Copy link
Contributor Author

@karol-bisztyga @jakub-gonet please re-check I have made some changes for this PR ✌️ ☃️

src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
@karol-bisztyga
Copy link
Contributor

For me, it looks OK. @jakub-gonet?

Copy link
Member

@jakub-gonet jakub-gonet left a comment

Choose a reason for hiding this comment

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

Left a few comments.

src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
src/reanimated2/interpolation.js Show resolved Hide resolved
src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
react-native-reanimated.d.ts Outdated Show resolved Hide resolved
Copy link
Member

@jakub-gonet jakub-gonet left a comment

Choose a reason for hiding this comment

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

LGTM, left one comment.

src/reanimated2/interpolation.js Outdated Show resolved Hide resolved
@karol-bisztyga karol-bisztyga merged commit a6ebe62 into software-mansion:master Jan 13, 2021
@Roka20012
Copy link
Contributor Author

@karol-bisztyga thanks for merging!❤️

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.

interpolate: Extrapolate left/right
5 participants