From e8859661955b796ee5022e85845dc6cbbd2fa2d8 Mon Sep 17 00:00:00 2001
From: Mark Liversedge <liversedge@gmail.com>
Date: Mon, 13 Jul 2020 14:43:06 +0100
Subject: [PATCH] Fix User Pie Chart Labels and Colors

.. bug not clearing labels before setting and now uses a gradient
   based upon the color selected by the user.
---
 src/Charts/UserChart.cpp | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/src/Charts/UserChart.cpp b/src/Charts/UserChart.cpp
index 2c3753c6b..3177c6f38 100644
--- a/src/Charts/UserChart.cpp
+++ b/src/Charts/UserChart.cpp
@@ -163,9 +163,22 @@ UserChart::setRide(RideItem *item)
         series.yseries = ucd->y.asNumeric();
 
         // pie charts need labels
-        if (chartinfo.type == GC_CHART_PIE || GC_CHART_BAR)
+        if (chartinfo.type == GC_CHART_PIE || chartinfo.type == GC_CHART_BAR) {
+            series.labels.clear();
             for(int i=0; i<ucd->x.asString().count(); i++) series.labels << ucd->x.asString()[i];
 
+            series.colors.clear();
+            QColor min=QColor(series.color);
+            QColor max=GCColor::invertColor(GColor(CPLOTBACKGROUND));
+            for(int i=0; i<series.labels.count(); i++) {
+                QColor color = QColor(min.red() + (double(max.red()-min.red()) * (i/double(series.labels.count()))),
+                              min.green() + (double(max.green()-min.green()) * (i/double(series.labels.count()))),
+                              min.blue() + (double(max.blue()-min.blue()) * (i/double(series.labels.count()))));
+
+                series.colors << color.name();
+            }
+        }
+
         // data now generated so can add curve
         chart->addCurve(series.name, series.xseries, series.yseries, series.xname, series.yname,
                         series.labels, series.colors,