-
Notifications
You must be signed in to change notification settings - Fork 7
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
Browser Support #6
Comments
Good point. That's nice it worked. |
Attaching the image to the DOM is a clever workaround for some use cases. However, I'm using images from drag-n-drop, and then converting those to 3D surfaces. This is part of the JSCAD project; www.jscad.xyz Good point about the async nature of the decoding. I'm not sure that's 100% required. Here's the code that produces the 3D surface.
P.S. Yeah. We have NODE.js package support inside the browser. |
@dy Here's another GREAT REASON to publish a web-ready package. This is really cool. |
I’d be glad to add a PR for this. What do you think? |
Sure! I am sorry - not fully understood the issue - hope the change isn't going to be large. |
See PR #7 |
Note that similar modifications can be reproduce to https://github.com/dy/image-encode With |
I was able to use this library in the browser by adding a build script based on browserify.
The standalone library can then be used within the browsers.
Just wondering... are you planning on adding this?
The text was updated successfully, but these errors were encountered: