Skip to content

Introducing requestAnimationFrame for effecincy#10

Open
maxpert wants to merge 1 commit into
madrobby:masterfrom
maxpert:master
Open

Introducing requestAnimationFrame for effecincy#10
maxpert wants to merge 1 commit into
madrobby:masterfrom
maxpert:master

Conversation

@maxpert
Copy link
Copy Markdown

@maxpert maxpert commented Apr 26, 2011

requestAnimationFrame is known for giving you native animation framing from browser thus allowing benefits like device native framerates and battery saving; this patch makes emile requestAnimationFrame enabled making it even more freaking cool!

@didxga
Copy link
Copy Markdown

didxga commented Apr 27, 2011

Cool, Thumb Up.

@cmawhorter
Copy link
Copy Markdown

Nice

@peterp
Copy link
Copy Markdown

peterp commented Sep 22, 2011

Agreed.

@zorker
Copy link
Copy Markdown

zorker commented Jan 18, 2012

Actually this does not work if you have multiple animations running parallel.
Basically what happens if you have 2 animations running all the time (image tweens with random images for example) and additionally a third element with onmouseover, onclick and onmouseout event (animation on each event). If you hover the third element it will start the animation for the third element but all other elements freeze. Seems like only one animation is handled at a time.

If you got a workaround for that please let me know. Currently trying to fiddle that out myself.

@maxpert
Copy link
Copy Markdown
Author

maxpert commented Jan 18, 2012

Would be trying to reproduce your bug

@zorker
Copy link
Copy Markdown

zorker commented Jan 18, 2012

I fixed it for me. Gonna post my fork in a minute.

Here we go: https://github.com/zorker/emile/blob/master/emile.js

I added opacity handling and animation processing aswell.

@getdave
Copy link
Copy Markdown

getdave commented Aug 9, 2013

@madrobby Just checking in to see whether this ever got merged. Don't think so from the look of things.

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.

6 participants