@datarailsshared/dr_renderer 1.2.194-dragons → 1.2.196-dragons

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@datarailsshared/dr_renderer",
-  "version": "1.2.194-dragons",
+  "version": "1.2.196-dragons",
   "description": "DataRails charts and tables renderer",
   "keywords": [
     "datarails",

package/src/highcharts_renderer.js

@@ -1343,7 +1343,7 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
         const colorOptions = opts.breakdown_options.colors;
         const chart_series = [];
         const row_n_keys = pivotData.getRowKeys();
-        const col_n_keys = lodash.map(lodash.keys(pivotData.colTotals), key => [key]);
+        const col_n_keys = pivotData.getColKeys();
         const hasBreakdownValues = lodash.get(opts, 'breakdown_options.values.breakdown.length');
 
         let resultObject = {
@@ -1360,16 +1360,26 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
         };
         resultObject = highchartsRenderer.getDataLabelsOptions(additionOptions, resultObject);
 
-        let previousTotalValue = 0;
         lodash.forEach(col_n_keys, function(col_n_value, col_index) {
 
             const totalColumnValue = pivotData.colTotals[col_n_value].sum;
-            if (col_index) {
+            const nextTotalColumnKey = col_n_keys[col_index + 1];
+            const nextTotalColumnValue = nextTotalColumnKey ? pivotData.colTotals[nextTotalColumnKey].sum : null;
+            resultObject.data.push({
+                y: totalColumnValue,
+                name: lodash.unescape(col_n_value).replace('DR_Others', highchartsRenderer.getOthersName(opts)),
+                isSum: !!col_index,
+                isTotal: true,
+                color: colorOptions.total,
+            });
+
+            if (col_index !== col_n_keys.length - 1) {
                 lodash.forEach(row_n_keys, function (row_n_value) {
                     const agg = pivotData.getAggregator(row_n_value, col_n_value);
                     let val = hasBreakdownValues
                         ? agg.value()
-                        : totalColumnValue - previousTotalValue;
+                        : nextTotalColumnValue - totalColumnValue;
+
                     val = $.isNumeric(val) ? parseFloat(val) : 0;
                     if (val) {
                         const tmoobj = {};
@@ -1390,14 +1400,6 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
                     }
                 });
             }
-            previousTotalValue = totalColumnValue;
-            resultObject.data.push({
-                y: totalColumnValue,
-                name: lodash.unescape(col_n_value).replace('DR_Others', highchartsRenderer.getOthersName(opts)),
-                isSum: !!col_index,
-                isTotal: true,
-                color: colorOptions.total,
-            });
         });
 
         chart_series.push(resultObject);
@@ -3175,6 +3177,8 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
             ? opts.chartOptions
             : highchartsRenderer.getDefaultValueForChart(highchartsRenderer.CHART_TYPES.WATERFALL_BREAKDOWN);
 
+        pivotData.colKeys = lodash.map(lodash.keys(pivotData.colTotals), key => [key]);
+
         chartOptions.chart = {
             type: 'waterfall',
             zoomType: additionOptions && additionOptions.chart && additionOptions.chart.zoom_type ? additionOptions.chart.zoom_type : 'None',
@@ -8662,11 +8666,10 @@ let getHighchartsRenderer = function ($, document, Highcharts, default_colors, h
     }
 
     highchartsRenderer.getBreakdownXAxisLabelFormatter = function(chartSeries) {
-        const styleForCategoryLabel = 'font-weight: 700; font-size: 12px;';
-        const styleForBreakdownLabel = 'font-weight: 600; color: #6f6f6f; font-size: 11px;';
+        const styleForBreakdownLabel = 'font-weight: normal; color: #51566f;';
         return function() {
             const isTotal = lodash.some(chartSeries[0].data, point => point.name === this.value && point.isTotal);
-            return `<span style="${ isTotal ? styleForCategoryLabel : styleForBreakdownLabel }">${ this.value }</span>`;
+            return `<span style="${ isTotal ? '' : styleForBreakdownLabel }">${ this.value }</span>`;
         }
     }