reshuffling

monfera · monfera · commit c252e17ee9c8 · 2017-09-29T16:19:24.000+02:00
diff --git a/src/traces/table/plot.js b/src/traces/table/plot.js
@@ -745,13 +745,12 @@ function updateYPositionMaker(columnBlock, element, tableControlView, d) {
 }
 
 function setCellHeightAndPositionY(columnCell) {
-    // fixme second perf bottleneck
     columnCell
         .attr('transform', function(d) {
+            var headerHeight = d.rowBlocks[0].auxiliaryBlocks.reduce(function(p, n) {return p + rowsHeight(n, Infinity)}, 0);
             var l = getBlock(d);
             var rowAnchor = rowsHeight(l, d.key);
             var rowOffset = firstRowAnchor(d.rowBlocks, l.key) + rowAnchor - firstRowAnchor(d.rowBlocks, d.page);
-            var headerHeight = d.rowBlocks[0].auxiliaryBlocks.reduce(function(p, n) {return p + rowsHeight(n, Infinity)}, 0);
             var yOffset = rowOffset + headerHeight;
             return 'translate(0 ' + yOffset + ')';
         })
@@ -761,7 +760,7 @@ function setCellHeightAndPositionY(columnCell) {
 
 function firstRowAnchor(blocks, p) {
     var pTop = 0;
-    for(var i = 0; i <= p - 1; i++) {
+    for(var i = 0; i < p; i++) {
         pTop += rowsHeight(blocks[i], Infinity);
     }
     return pTop;
