handle percentage case for tooltips (#2570)

apache · Apr 11, 2017 · ab0bc5a · ab0bc5a
1 parent 412634c
commit ab0bc5a
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/superset/assets/visualizations/nvd3_vis.js b/superset/assets/visualizations/nvd3_vis.js
@@ -211,6 +211,13 @@ function nvd3Vis(slice, payload) {
         chart.labelThreshold(0.05)  // Configure the minimum slice size for labels to show up
           .labelType(fd.pie_label_type);
         chart.cornerRadius(true);
+
+        if (fd.pie_label_type === 'percent') {
+          let total = 0;
+          payload.data.forEach((d) => { total += d.y; });
+          chart.tooltip.valueFormatter(d => `${((d / total) * 100).toFixed()}%`);
+        }
+
         break;
 
       case 'column':