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

Define enumerations for animation #566

Merged

Conversation

shoheiyokoyama
Copy link
Contributor

@shoheiyokoyama shoheiyokoyama commented Oct 25, 2016

#565

into development

Copy link
Member

@daniel-jonathan daniel-jonathan left a comment

Choose a reason for hiding this comment

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

I will make those minor changes.

Aside from the changes, wonderful work and thank you :)

@@ -55,6 +55,35 @@ public func AnimationFillModeToValue(mode: AnimationFillMode) -> String {
}
}

@objc(AnimationTimingFunction)
public enum AnimationTimingFunction: Int {
case liner
Copy link
Member

Choose a reason for hiding this comment

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

I think you meant to write linear.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, I'm sorry 😞

Copy link
Member

Choose a reason for hiding this comment

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

no worries :) You did great work. Thank you!

case easeIn
case easeOut
case easeInEaseOut
case systemDefault
Copy link
Member

Choose a reason for hiding this comment

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

this can be default

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for your advice!

@daniel-jonathan daniel-jonathan merged commit c97e64b into CosmicMind:development Oct 25, 2016
@daniel-jonathan
Copy link
Member

Your PR inspired me to refactor the Animation class, thank you!

486cab2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants