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

Increase performance & make it compatible with more browsers #1

Merged
merged 22 commits into from Jun 7, 2019

Conversation

Projects
None yet
2 participants
@lovasoa
Copy link
Contributor

commented May 31, 2019

Switch from using 1 pixel fillRect on an offscreen canvas to using a typed array on the workers. This improves rendering performance, and makes this work on non-chrome browsers.
Tested on chrome, firefox and safari.

@ScionOfBytes
Copy link
Owner

left a comment

@lovasoa Thank you so much for the contribution! I'm gonna merge these and then make a few stylistic changes, hope you don't mind. Could you explain some of the improvements you made? They go right over my head. I wanna add an updated section to the blog post explaining your changes and how they help performance.

Show resolved Hide resolved js/compy-stuff.js Outdated
Show resolved Hide resolved js/painter.js Outdated
Show resolved Hide resolved js/sketch.js Outdated
Show resolved Hide resolved js/painter.js Outdated
Show resolved Hide resolved js/compy-stuff.js

@ScionOfBytes ScionOfBytes merged commit 2ad7abe into ScionOfBytes:master Jun 7, 2019

2 of 5 checks passed

Header rules - smooth-mandelbrot-scionofbytes No header rules processed
Details
Pages changed - smooth-mandelbrot-scionofbytes 2 new files uploaded
Details
Redirect rules - smooth-mandelbrot-scionofbytes No redirect rules processed
Details
Mixed content - smooth-mandelbrot-scionofbytes No mixed content detected
Details
netlify/smooth-mandelbrot-scionofbytes/deploy-preview Deploy preview ready!
Details
Show resolved Hide resolved js/acolyte.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.