Add AMD (Require.js) + CommonJS (browserify) wrapper

[thibaudcolas]

Hi,

here's a PR to add support for two module loaders: Require.js (AMD modules, define) and browserify (Node-ish/CommonJS, module.exports). It should help with the following issues: #63 and #73.

The UMD wrapper is taken from https://github.com/eduardolundgren/gulp-umd (see also https://github.com/umdjs/umd). I changed it a little bit to accomodate the following browserify issue (use window || this for root instead of this).

The AMD / Require.js scenario is fully working, and I added a test case for it (http://127.0.0.1:8000/tests/?amd=1).

The CommonJS / browserify story is a little different. It's also fully working, but there are caveats:

• To take full advantage of CommonJS compatibility InstantClick would need to be available in npm: it means adding a package.json, and if possible publishing it on the npm repository. After that, users will be able to require('instantclick') instead of require('../../vendor/instantclick-x.x.x/instantclick').
• Modules are statically bundled by browserify prior to being served on a page: a test case for browserify would need a build system in place, which is out of scope for this PR. On the flip side, I did tests on my computer and uploaded the resulting browserify bundle as a gist. It works very well.

Let me know if it suits you!

[dieulot]

Thanks for this. Currently I’m focused on getting InstantClick out of its hiatus (= shipping InstantClick 3.1), I’ll take a deeper look at this once the project is moving again.

[dieulot]

So I’ve looked a bit at Require and Browserify and I see more disadvantages than advantages in including them, so I don’t plan to merge, sorry. More details in #63 and #73.

[thibaudcolas]

Hey there,

no worries, it's already cool that you took some time to look at it. I answered in #63 and I'll stop here.

Take care.

[egoist]

Just FYI, I built a UMD version of this library two years ago for you to use in a modern bundler:

https://github.com/egoist/instantclick

/related: #156 #63 #73