Browserify-able npm module

[danreeves]

The current dist errors when required through browserify.

Browserify is not smart and tries resolving every require() it comes across, even the ones which have already been resolved (i.e. this file). This shouldn't be happening since the dist build has the standalone option. Using the .min file fixes this but it isn't included in the npm distribution.

According to this comment on a browserify issue derequire was removed from the standalone option, and it needs to be added as an additional transform.

[jhchen]

To clarify you are trying to run command line browserify on a file that required Quill?

[danreeves]

Yes; sorry, should have been more clear.

I am using browserify (through gulp) to build my project and Quill is a dependency in my project.

Steps to reproduce would be:

• npm install quill
• browserify app.js > bundle.js

Where app.js:

var Quill = require('quill');

The browserify errors are all missing dependencies that correspond to the requires inside quill.js. It gets to that point so I don't think the issues is the browser field in package.json as mentioned in issue #272.

If you see this issue it can be resolved by adding an additional step to the browserify build. I'm afraid I'm not a grunt or coffeescript guy, else I would have given a pull request a go.

Thanks.

[jhchen]

No worries those two linked issues were very helpful in getting to the bottom of this.

[danreeves]

This is great. Thanks a lot!