-
Notifications
You must be signed in to change notification settings - Fork 82
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
Support for AMD by using UMD or similar #21
Comments
Wow, that looks gnarly. Browsers :( I'd be open to a pull request that added support for browser-side amd, as long as it was also compatible with browserify. |
It´s not that bad. Just remove all the comments (which no one reads anyways) and it´s only a few lines of code ;) Okay, I will have a look at it. And try browserify. Cheers |
I did the browserify stuff properly (UMD-style) and added a demo in that last PR. |
It might make sense to add an empty gh-pages branch and put clientside.html in there as index.html, and the dist folder, then people can use gh-pages like a CDN, and you have a nice live demo, like so. |
Could you maybe add support for AMD by using the Universal Module Definition which supports CommonJS/NodeJS, AMD and good old globals?
Read about it here: https://github.com/umdjs/umd
Example implementation: https://github.com/umdjs/umd/blob/master/returnExportsGlobal.js
Thank you
Regards
The text was updated successfully, but these errors were encountered: