From a43946af0081a3e368c23a3178434c393a322fce Mon Sep 17 00:00:00 2001
From: Kyle Doherty <kyle.l.doherty@gmail.com>
Date: Tue, 12 Jul 2016 15:31:41 -0700
Subject: [PATCH] use color harmony for more than 5 slices
---
frontend/src/metabase/visualizations/PieChart.jsx | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/frontend/src/metabase/visualizations/PieChart.jsx b/frontend/src/metabase/visualizations/PieChart.jsx
index a478e4b6b99..cbdf6ffc1a9 100644
--- a/frontend/src/metabase/visualizations/PieChart.jsx
+++ b/frontend/src/metabase/visualizations/PieChart.jsx
@@ -59,7 +59,8 @@ export default class PieChart extends Component {
let total = data.rows.reduce((sum, row) => sum + row[1], 0);
- let sliceColors = Object.values(colors.normal);
+ // use standard colors for up to 5 values otherwise use color harmony to help differentiate slices
+ let sliceColors = Object.values(data.rows.length > 5 ? colors.harmony : colors.normal);
let [slices, others] = _.chain(data.rows)
.map(([key, value], index) => ({
--
GitLab