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

[AnimationExperimental] Fixed CATransaction completion block invoke immediately #1400

Closed
wants to merge 1 commit into from

Conversation

Shuangzuan
Copy link
Contributor

CATransaction Class Reference

In Tasks > Getting and Setting Completion Block Objects > Discussion:

The completion block object that is guaranteed to be called (on the main thread) as soon as all animations subsequently added by this transaction group have completed (or have been removed.) If no animations are added before the current transaction group is committed (or the completion block is set to a different value,) the block will be invoked immediately.

@ide
Copy link
Contributor

ide commented May 26, 2015

cc @nicklockwood

@facebook-github-bot
Copy link
Contributor

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at [email protected]. Thanks!

@facebook-github-bot
Copy link
Contributor

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@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 May 26, 2015
@brentvatne brentvatne changed the title Fixed CATransaction completion block invoke immediately [AnimationExperimental] Fixed CATransaction completion block invoke immediately Jun 1, 2015
@sahrens
Copy link
Contributor

sahrens commented Jun 11, 2015

Looks like you have two totally different commits stacked in this PR - can you move your push notification change to new PR?

@Shuangzuan
Copy link
Contributor Author

@sahrens
OK. Finished.

@a2 a2 closed this Jun 18, 2015
ayushjainrksh pushed a commit to MLH-Fellowship/react-native that referenced this pull request Jul 2, 2020
mganandraj pushed a commit to mganandraj/react-native that referenced this pull request Sep 29, 2022
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.

5 participants