Merge pull request #213 from PanSpagetka/fix-pie-chart-click-index

Fix Chart pie clicking when part of it is hidden
ManageIQ · Jan 23, 2017 · 410462e · 410462e
2 parents c1bbf88 + 126d692
commit 410462e
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/app/assets/javascripts/miq_c3.js b/app/assets/javascripts/miq_c3.js
@@ -25,8 +25,10 @@ function load_c3_chart(data, chart_id, height) {
   var generate_args = chartData(data.miqChart, data, { bindto: "#" + chart_id, size: {height: height}})
 
   generate_args.data.onclick = function (data, _i) {
-    var seriesIndex = _.findIndex(generate_args.data.columns, function(col) { return col[0] == data.id; }) - 1
-    var pointIndex = data.index;
+    var index = _.findIndex(generate_args.data.columns, function(col) { return col[0] == data.id; });
+    // when not Pie/Donut chart, first column doesn't contain actual data.
+    var seriesIndex = _.contains(['Pie', 'Donut'], generate_args.miqChart) ? index : index - 1;
+    var pointIndex = _.contains(['Pie', 'Donut'], generate_args.miqChart) ? index : data.index;
     var value = data.value;
 
     var parts = chart_id.split('_'); // miq_chart_candu_2