clean up
authorDan Vanderkam <danvk@google.com>
Wed, 10 Nov 2010 16:08:59 +0000 (11:08 -0500)
committerDan Vanderkam <danvk@google.com>
Wed, 10 Nov 2010 16:08:59 +0000 (11:08 -0500)
docs/index.html
dygraph.js

index fe36356..811b7b7 100644 (file)
@@ -853,7 +853,7 @@ perl -ne 'BEGIN{print "Month,Nominal,Real\n"} chomp; ($m,$cpi,$low,$close,$high)
 
         <tr>
           <td><strong>highlightCallback</strong></td>
-          <td><code>function(event, x, points)</code></td>
+          <td><code>function(event, x, points,row)</code></td>
           <td><code>null</code></td>
           <td>When set, this callback gets called every time a new point is highlighted. The parameters are the JavaScript mousemove event, the x-coordinate of the highlighted points and an array of highlighted points: <code>[ {name: 'series', yval: y-value}, &hellip; ]</code>
           <div class="tests">Tests: <a href="tests/callback.html">callback</a> <a href="tests/crosshair.html">crosshair</a> </div>
index 1f36f1c..5cfe2f4 100644 (file)
@@ -1214,23 +1214,22 @@ Dygraph.prototype.mouseMove_ = function(event) {
 };
 
 /**
- * Transforms layout_.points index into data row number
+ * Transforms layout_.points index into data row number.
  * @param int layout_.points index
- * @return int row number
+ * @return int row number, or -1 if none could be found.
  * @private
  */
 Dygraph.prototype.idxToRow_ = function(idx) {
-       if(idx<0)return -1;
-
-       for (var i in this.layout_.datasets) {
-               if (idx < this.layout_.datasets[i].length) {
-                       return this.boundaryIds_[0][0]+idx;
-               }
-               idx-=this.layout_.datasets[i].length;
-       }
-       return -1;
-}
+  if (idx < 0) return -1;
 
+  for (var i in this.layout_.datasets) {
+    if (idx < this.layout_.datasets[i].length) {
+      return this.boundaryIds_[0][0]+idx;
+    }
+    idx -= this.layout_.datasets[i].length;
+  }
+  return -1;
+};
 
 /**
  * Draw dots over the selectied points in the data series. This function