Merge pull request #241 from alison985/204_fix_unable_to_save

make pie chart colors work w/o having to choose each color.
mozilla · Sep 1, 2017 · 3df714d · 3df714d
2 parents 17ba8e7 + 6e9170c
commit 3df714d
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/client/app/visualizations/chart/plotly.js b/client/app/visualizations/chart/plotly.js
@@ -273,7 +273,12 @@ const PlotlyChart = () => {
             series.data.forEach((row) => {
               plotlySeries.values.push(row.y);
               plotlySeries.labels.push(hasX ? row.x : `Slice ${index}`);
-              plotlySeries.marker.colors.push(scope.options.seriesOptions[hasX ? row.x : `Slice ${index}`].color);
+              if (scope.options.seriesOptions[hasX ? row.x : `Slice ${index}`] === undefined) {
+                plotlySeries.marker.colors.push(getColor(index));
+                index += 1;
+              } else {
+                plotlySeries.marker.colors.push(scope.options.seriesOptions[hasX ? row.x : `Slice ${index}`].color);
+              }
             });
 
             scope.data.push(plotlySeries);