[perf-web] Fix ranges for multiple columns of target data



commit b3537990bc7b6eac7e0d64a8f925d96f64737146
Author: Owen W. Taylor <otaylor fishsoup net>
Date:   Sun Jun 29 11:01:39 2014 -0400

    Fix ranges for multiple columns of target data
    
    Units got confused when there were multiple columns.

 static/main.js |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)
---
diff --git a/static/main.js b/static/main.js
index 1d03c58..7f24912 100644
--- a/static/main.js
+++ b/static/main.js
@@ -742,6 +742,7 @@ TargetTable.prototype.prepareColumns = function() {
 
     var columns = [];
 
+    var displayUnits;
     this.head.find( "th" ).each(function(index, element) {
         if (!element.hasAttribute('data-target'))
             return;
@@ -751,14 +752,14 @@ TargetTable.prototype.prepareColumns = function() {
             var values = metricData[target];
             var column = new TableColumn(values);
             column.prepareRange(min, max, units);
-            units = column.units;
+            displayUnits = column.units;
             columns.push(column);
         } else {
             columns.push(null);
         }
     });
 
-    $( '#metricUnits' ).text(units);
+    $( '#metricUnits' ).text(displayUnits);
 
     return columns;
 }


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]