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

Add details for Animated.forkEvent #403

Merged
merged 1 commit into from Jun 15, 2018

Conversation

Projects
None yet
3 participants
@slorber
Contributor

slorber commented Jun 11, 2018

it was unclear to me if event could actually be a considered as an opaque structure, ie if it would work as expected for any kind of thing the user can actually provide as a onScroll listener for example.

It should be clear that it's safe to call with null or regular js listener

cc @sahrens

Add details for forkEvent
it was unclear to me if `event` could actually be a considered as an opaque structure, ie if it would work as expected for any kind of thing the user can actually provide as a `onScroll` listener for example.

It should be clear that it's safe to call with null or regular js listener

cc @sahrens

@slorber slorber changed the title from Add details for forkEvent to Add details for Animated.forkEvent Jun 11, 2018

@react-native-bot

This comment has been minimized.

react-native-bot commented Jun 11, 2018

Deploy preview for react-native ready!

Built with commit e78cfa5

https://deploy-preview-403--react-native.netlify.com

Changes to docs/ are reflected in the next "master" version.

Thank you for your contributions.

How to ContributeDocumentation Sources

@hramos hramos merged commit b2d9cce into facebook:master Jun 15, 2018

2 checks passed

ci/circleci: checkout_code Your tests passed on CircleCI!
Details
ci/circleci: test_website Your tests passed on CircleCI!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment