CommonJS and browserify support

alexstrat commented 11 years ago

Hi,

I tried to use backbone-forms with browserify (~ CommonJS bundler), but I got some issues:

browserify uses main field of package.json to find the file to bundle. However, here, the main field points to the AMD bundle. So i added the browser field that browserify understands.
in nodejs's CommonJS module – and, thus, in browserify – the global this does not refer to the global object. Then, root is not what it's expected to be. I changed that to explicitly pass window || global || this (in browserify, global = window)
Backbone.Form was not exported as commonJS module: fixed that. var Form = require('backbone-forms');

And with that, it seems to be ok.

powmedia commented 11 years ago

Thanks - releasing as npm package v0.12.0

powmedia commented 11 years ago

Actually will be v0.13.0

olasfar commented 11 years ago

Looks good, thanks a lot @alexstrat. @powmedia, when the v0.13.0 will be available?

Thx,

powmedia commented 11 years ago

It's available now, tagged on github and published on npm

On 3 Nov 2013, at 15:09, olasfar notifications@github.com wrote:

Thx,

powmedia / backbone-forms