Load dygraph-combined when loading from NPM
authorAlden Daniels <alden@aldendaniels.me>
Mon, 20 Apr 2015 16:25:29 +0000 (11:25 -0500)
committerDan Vanderkam <danvdk@gmail.com>
Tue, 26 May 2015 20:56:32 +0000 (16:56 -0400)
Today, when loading dygraphs from NPM via browserify you need to include:

```javascript
var Dygraph  = require('dygraphs/dygraph-combined.js');
```

Being able to do this:
```javascript
var Dygraph  = require('dygraphs');
```

Is preferable.

Use "dev" combined version

Minification should be handled by each app's build system.

Export Dygraph

Fix broken tests when module is not defined

dygraph.js
package.json

index eab46a2..3569ee1 100644 (file)
@@ -3801,6 +3801,10 @@ Dygraph.addAnnotationRule = function() {
   console.warn("Unable to add default annotation CSS rule; display may be off.");
 };
 
+if (typeof exports === "object" && typeof module !== "undefined") {
+  module.exports = Dygraph;
+}
+
 return Dygraph;
 
 })();
index 84948e9..bb758fd 100644 (file)
@@ -2,7 +2,7 @@
   "name": "dygraphs",
   "version": "1.1.0",
   "description": "dygraphs is a fast, flexible open source JavaScript charting library.",
-  "main": "dygraph.js",
+  "main": "dygraph-combined-dev.js",
   "directories": {
     "doc": "docs",
     "test": "tests"