Publish to npm about alertifyjs HOT 9 CLOSED

I'm not sure if its useful to publish to npm, as this is not a node module. It has pacakage.json mainly for installing dev. dependencies.

Don't know how Browserify fits here, its supposed to work the other way around (making node modules work in the browser)!

from alertifyjs.

velocity-animate and cropper are similarly dedicated to browsers but are in npm.
Libraries like jQuery, moment, lodash, postal, q, and string that work on both client and server explicitly support using the npm installed package in the browser.

Browserify doesn't simply make node modules work in the browser. It lets you write your code in CommonJS/Node.js style using require() and local module scope. Then when built wraps it up and automatically wraps up and bundles the code for any modules that it depends on via require().

If I were to npm install someclientlib --save and then add var SomeClientLib = require('someclientlib'); and rebuild my client code, browserify would include the library and my module could just start using it.

The standard source for these browserify looks for in is npm (or rather node_modules/) where package.json can have an optional tiny bit extra if the package should use a different set of modules on the client.

To include something not in npm I have to mess around, either doing an ugly relative require or adding a hack to my package.json to tell it to load a certain file when I write require('foobar').

from alertifyjs.

Also, https://www.npmjs.com/package/alertify

from alertifyjs.

Though I do admit the "AMD and window support" section also needs an update to support browserify.

from alertifyjs.

Done.

from alertifyjs.

Something like:

    // AMD and window support
    if ( typeof module === 'object' && typeof module.exports === 'object' ) {
        module.exports = alertify;
    } else if ( typeof define === 'function' ) {
        define( [], function () {
            return alertify;
        } );
    } else if ( !window.alertify ) {
        window.alertify = alertify;
    }

from alertifyjs.

I'll make a pull request for the other tweaks for full npm/browserify support.

from alertifyjs.

That would be great!

from alertifyjs.

Published in 1.2.0

Thanks!

from alertifyjs.