Render Zero Values in Percent Charts Properly

src/kibana/components/agg_response/point_series/_tooltip_formatter.js

@@ -16,12 +16,20 @@ define(function (require) {
       while ((result = result.$parent) && result.aggConfig) {
         var agg = result.aggConfig;
         var value = result.value;
-        if (agg === datum.aggConfigResult.aggConfig && datum.yScale != null) value *= datum.yScale;
 
-        details.push({
+        var detail = {
           value: agg.fieldFormatter()(value),
           label: agg.makeLabel()
-        });
+        };
+
+        if (agg === datum.aggConfigResult.aggConfig) {
+          detail.percent = event.percent;
+          if (datum.yScale != null) {
+            detail.value = agg.fieldFormatter()(value * datum.yScale);
+          }
+        }
+
+        details.push(detail);
       }
 
       $tooltipScope.$apply();

src/kibana/components/vislib/lib/dispatch.js

@@ -43,19 +43,7 @@ define(function (require) {
       var color = handler.data.color;
       var isPercentage = (handler._attr.mode === 'percentage');
 
-      if (isSeries) {
-        // Find object with the actual d value and add it to the point object
-        var object = _.find(series, { 'label': d.label });
-        d.value = +object.values[i].y;
-
-        if (isPercentage) {
-
-          // Add the formatted percentage to the point object
-          d.percent = (100 * d.y).toFixed(1) + '%';
-        }
-      }
-
-      return {
+      var eventData = {
         value: d.y,
         point: datum,
         datum: datum,
@@ -69,6 +57,19 @@ define(function (require) {
         e: d3.event,
         handler: handler
       };
+
+      if (isSeries) {
+        // Find object with the actual d value and add it to the point object
+        var object = _.find(series, { 'label': d.label });
+        eventData.value = +object.values[i].y;
+
+        if (isPercentage) {
+          // Add the formatted percentage to the point object
+          eventData.percent = (100 * d.y).toFixed(1) + '%';
+        }
+      }
+
+      return eventData;
     };
 
     /**

src/kibana/components/vislib/visualizations/column_chart.js

@@ -146,6 +146,13 @@ define(function (require) {
           return Math.abs(yScale(d.y0 + d.y) - yScale(d.y0));
         }
 
+        // Due to an issue with D3 not returning zeros correctly when using
+        // an offset='expand', need to add conditional statement to handle zeros
+        // appropriately
+        if (d._input.y === 0) {
+          return 0;
+        }
+
         // for split bars or for one series,
         // last series will have d.y0 = 0
         if (d.y0 === 0 && yMin > 0) {