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

Adding support for the animator proxy on OS X #41

Merged
merged 2 commits into from
Jan 14, 2014

Conversation

pfandrade
Copy link

Enabled using the animator proxy on OS X (issue #39)

@pfandrade
Copy link
Author

I have started an Example Mac project on my fork:master if you want to have a quick look at this working on the Mac. If I have time to beef up the example project I'll send a pull request for it separately.

{
for (id<MASConstraint> constraint in self.childConstraints) {
#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Wunused-value"
Copy link
Contributor

Choose a reason for hiding this comment

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

If you use [constraint animator] you will avoid having to ignore the warning

@cloudkite
Copy link
Contributor

@pfandrade looking great! Just a couple of minor things

@cloudkite
Copy link
Contributor

Also mac example proj however basic would be great has been on todo list for a while 👍

@pfandrade
Copy link
Author

Added the style fixes.

As for the Mac example, I'll clean it up, and add a couple more examples before issuing the pull request. Right now it's just a quick way to test this animator proxy code.

@cloudkite
Copy link
Contributor

Nice! Thanks again for the contribution 👍

cloudkite added a commit that referenced this pull request Jan 14, 2014
Adding support for the animator proxy on OS X
@cloudkite cloudkite merged commit 9806c36 into SnapKit:master Jan 14, 2014
dan-rodrigues pushed a commit to dan-rodrigues/Masonry that referenced this pull request Oct 18, 2014
Adding support for the animator proxy on OS X
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.

2 participants