demo chart renders correctly with combined; interaction broken
authorDan Vanderkam <danvdk@gmail.com>
Sun, 1 Sep 2013 16:53:08 +0000 (12:53 -0400)
committerDan Vanderkam <danvdk@gmail.com>
Sun, 1 Sep 2013 16:53:08 +0000 (12:53 -0400)
dygraph.js
generate-combined.sh
plugins/annotations.js
plugins/axes.js
plugins/chart-labels.js
plugins/grid.js
plugins/legend.js
plugins/range-selector.js

index d9695a2..c413166 100644 (file)
@@ -281,6 +281,7 @@ Dygraph.prototype.init_ = function(div, file, opt_attrs) {
       }
     }
   }
+  window.console.log(this.eventListeners_);
 
   this.createDragInterface_();
 
index 54cfd9e..9839502 100755 (executable)
@@ -46,7 +46,7 @@ Copyright () {
 CatMinified () {
   Copyright
   (CatSources | grep -v '@license') \
-  | java -jar ../../closure-compiler-read-only/build/compiler.jar --js ../../closure-library-read-only/closure/goog/base.js --js - --js dygraph-exports.js --compilation_level ADVANCED_OPTIMIZATIONS --warning_level VERBOSE --externs dygraph-externs.js --externs gviz-api.js --output_wrapper='(function() {%output%})();'
+  | java -jar ../../closure-compiler-read-only/build/compiler.jar --js ../../closure-library-read-only/closure/goog/base.js --formatting PRETTY_PRINT --js - --js dygraph-exports.js --compilation_level ADVANCED_OPTIMIZATIONS --warning_level VERBOSE --externs dygraph-externs.js --externs gviz-api.js --output_wrapper='(function() {%output%})();'
   # Add this for debugging:
   #  --formatting PRETTY_PRINT 
 
index e518fe3..f6fcb87 100644 (file)
@@ -33,8 +33,8 @@ annotations.prototype.toString = function() {
 
 annotations.prototype.activate = function(g) {
   return {
-    clearChart: this.clearChart,
-    didDrawChart: this.didDrawChart
+    'clearChart': this.clearChart,
+    'didDrawChart': this.didDrawChart
   };
 };
 
index d1fe9b9..2ff2400 100644 (file)
@@ -43,9 +43,9 @@ axes.prototype.toString = function() {
 
 axes.prototype.activate = function(g) {
   return {
-    layout: this.layout,
-    clearChart: this.clearChart,
-    willDrawChart: this.willDrawChart
+    'layout': this.layout,
+    'clearChart': this.clearChart,
+    'willDrawChart': this.willDrawChart
   };
 };
 
index da231d8..a3c8e11 100644 (file)
@@ -25,9 +25,9 @@ chart_labels.prototype.toString = function() {
 
 chart_labels.prototype.activate = function(g) {
   return {
-    layout: this.layout,
+    'layout': this.layout,
     // clearChart: this.clearChart,
-    didDrawChart: this.didDrawChart
+    'didDrawChart': this.didDrawChart
   };
 };
 
index 1fb34fc..392aec6 100644 (file)
@@ -33,7 +33,7 @@ grid.prototype.toString = function() {
 
 grid.prototype.activate = function(g) {
   return {
-    willDrawChart: this.willDrawChart
+    'willDrawChart': this.willDrawChart
   };
 };
 
index 6684641..2877987 100644 (file)
@@ -102,11 +102,11 @@ legend.prototype.activate = function(g) {
   this.one_em_width_ = 10;  // just a guess, will be updated.
 
   return {
-    select: this.select,
-    deselect: this.deselect,
+    'select': this.select,
+    'deselect': this.deselect,
     // TODO(danvk): rethink the name "predraw" before we commit to it in any API.
-    predraw: this.predraw,
-    didDrawChart: this.didDrawChart
+    'predraw': this.predraw,
+    'didDrawChart': this.didDrawChart
   };
 };
 
@@ -138,6 +138,7 @@ legend.prototype.deselect = function(e) {
 };
 
 legend.prototype.didDrawChart = function(e) {
+  window.console.log('legend.didDrawChart');
   this.deselect(e);
 };
 
@@ -151,6 +152,7 @@ legend.prototype.didDrawChart = function(e) {
  * @private
  */
 legend.prototype.predraw = function(e) {
+  window.console.log('legend.predraw');
   // Don't touch a user-specified labelsDiv.
   if (!this.is_generated_div_) return;
 
index 5c9df58..59bcca4 100644 (file)
@@ -34,9 +34,9 @@ rangeSelector.prototype.activate = function(dygraph) {
     this.createInterface_();
   }
   return {
-    layout: this.reserveSpace_,
-    predraw: this.renderStaticLayer_,
-    didDrawChart: this.renderInteractiveLayer_
+    'layout': this.reserveSpace_,
+    'predraw': this.renderStaticLayer_,
+    'didDrawChart': this.renderInteractiveLayer_
   };
 };