Skip to content
This repository has been archived by the owner on Jul 21, 2023. It is now read-only.

RAF it Bro #10

Merged
merged 2 commits into from
Jul 15, 2015
Merged

RAF it Bro #10

merged 2 commits into from
Jul 15, 2015

Conversation

longoria
Copy link

@joerozek @JasonRose

Found this while browsing the fabricjs issue tracker:
fabricjs#1979 (comment)

Using requestAnimationFrame actually cuts grid page render time almost in half.

Here are some numbers, in seconds: (Command+A start, complete render stop)

Safari before:
8.00
9.00
8.36

Safari after:
4.55
4.46
4.45

Chrome before:
10.25
10.31
10.08

Chrome after:
5.86
5.96
5.83

@joerozek
Copy link

See Ben this is why you get all the performance issues. Because you're to dang good at them. Nice work Ben!

🍔 🍔 🍔 🎤 🎤

@gantaa
Copy link

gantaa commented Jul 14, 2015

I like how i'm still on this thread

On Tue, Jul 14, 2015 at 4:33 PM, Joe R notifications@github.com wrote:

See Ben this is why you get all the performance issues. Because you're to
dang good at them. Nice work Ben!

[image: 🍔] [image: 🍔] [image: 🍔] [image:
🎤] [image: 🎤]


Reply to this email directly or view it on GitHub
#10 (comment)
.

@joerozek
Copy link

LOL

@longoria
Copy link
Author

@gantaa ping pong?

longoria added a commit that referenced this pull request Jul 15, 2015
@longoria longoria merged commit c51b9a9 into master Jul 15, 2015
@longoria longoria deleted the raf-it-bro branch July 15, 2015 23:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants