-
Notifications
You must be signed in to change notification settings - Fork 25.6k
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
feat(animate): cross-browser compatibility #4243
Conversation
8b18607
to
600e938
Compare
let me know when it is ready for review |
51e5cd2
to
ab32c35
Compare
Ready for review /cc @robertmesserle |
|
||
//RequestAnimationFrame (IE9, Android 4.1, 4.2, 4.3) | ||
/*! @author Paul Irish */ | ||
/*! @source https://gist.github.com/paulirish/1579671 */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice if we can use a node module or a github repo, in case someone fixes some bugs on it, etc. No worries if you can't find any.
I thought this also broke during the traceur removal. Now I see that it is independent issue with animation, so I will let @robertmesserle do the final review. LGTM from my standpoint. |
ab32c35
to
5341267
Compare
LGTM |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Follow up of #4219, only the last 2 commits have to be reviewed.