From d6dee559f716b37b2c01778fd6de86f5744c89c4 Mon Sep 17 00:00:00 2001
From: a0z <a0z@noreply.codeberg.org>
Date: Tue, 17 Dec 2024 21:18:41 +0100
Subject: [PATCH] PAI fragment: update colors, replace piechart with gauge

---
 .../activities/charts/PaiChartFragment.java   | 108 +++++++-----------
 .../main/res/layout/fragment_pai_chart.xml    |  22 ++--
 app/src/main/res/values/colors.xml            |   3 +
 app/src/main/res/values/strings.xml           |   1 +
 4 files changed, 56 insertions(+), 78 deletions(-)

diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/charts/PaiChartFragment.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/charts/PaiChartFragment.java
index 0c4fbed42..b1a75e422 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/charts/PaiChartFragment.java
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/charts/PaiChartFragment.java
@@ -19,9 +19,11 @@ package nodomain.freeyourgadget.gadgetbridge.activities.charts;
 import android.graphics.Color;
 import android.os.Build;
 import android.os.Bundle;
+import android.util.TypedValue;
 import android.view.LayoutInflater;
 import android.view.View;
 import android.view.ViewGroup;
+import android.widget.ImageView;
 import android.widget.LinearLayout;
 import android.widget.TextView;
 
@@ -38,9 +40,6 @@ import com.github.mikephil.charting.data.BarData;
 import com.github.mikephil.charting.data.BarDataSet;
 import com.github.mikephil.charting.data.BarEntry;
 import com.github.mikephil.charting.data.ChartData;
-import com.github.mikephil.charting.data.PieData;
-import com.github.mikephil.charting.data.PieDataSet;
-import com.github.mikephil.charting.data.PieEntry;
 import com.github.mikephil.charting.formatter.ValueFormatter;
 
 import org.slf4j.Logger;
@@ -55,6 +54,7 @@ import java.util.Optional;
 
 import nodomain.freeyourgadget.gadgetbridge.GBApplication;
 import nodomain.freeyourgadget.gadgetbridge.R;
+import nodomain.freeyourgadget.gadgetbridge.activities.dashboard.GaugeDrawer;
 import nodomain.freeyourgadget.gadgetbridge.database.DBHandler;
 import nodomain.freeyourgadget.gadgetbridge.devices.DeviceCoordinator;
 import nodomain.freeyourgadget.gadgetbridge.devices.TimeSampleProvider;
@@ -69,7 +69,7 @@ public class PaiChartFragment extends AbstractChartFragment<PaiChartFragment.Pai
 
     protected Locale mLocale;
 
-    protected PieChart mTodayPieChart;
+    protected ImageView mGoalMinutesGauge;
     protected BarChart mWeekChart;
     protected TextView mDateView;
     protected TextView mLineToday;
@@ -98,8 +98,8 @@ public class PaiChartFragment extends AbstractChartFragment<PaiChartFragment.Pai
         LEGEND_TEXT_COLOR = DESCRIPTION_COLOR = GBApplication.getTextColor(requireContext());
         CHART_TEXT_COLOR = ContextCompat.getColor(requireContext(), R.color.secondarytext);
 
-        PAI_TOTAL_COLOR = ContextCompat.getColor(requireContext(), R.color.chart_deep_sleep_light);
-        PAI_DAY_COLOR = ContextCompat.getColor(requireContext(), R.color.chart_activity_dark);
+        PAI_TOTAL_COLOR = ContextCompat.getColor(requireContext(), R.color.chart_pai_weekly);
+        PAI_DAY_COLOR = ContextCompat.getColor(requireContext(), R.color.chart_pai_today);
     }
 
     @Override
@@ -116,7 +116,7 @@ public class PaiChartFragment extends AbstractChartFragment<PaiChartFragment.Pai
             });
         }
 
-        mTodayPieChart = rootView.findViewById(R.id.pai_chart_today);
+        mGoalMinutesGauge = rootView.findViewById(R.id.goal_minutes_gauge);
         mWeekChart = rootView.findViewById(R.id.pai_chart_week);
         mDateView = rootView.findViewById(R.id.pai_date_view);
         mLineToday = rootView.findViewById(R.id.pai_line_today);
@@ -142,7 +142,6 @@ public class PaiChartFragment extends AbstractChartFragment<PaiChartFragment.Pai
         }
 
         setupWeekChart();
-        setupTodayPieChart();
 
         // refresh immediately instead of use refreshIfVisible(), for perceived performance
         refresh();
@@ -150,21 +149,6 @@ public class PaiChartFragment extends AbstractChartFragment<PaiChartFragment.Pai
         return rootView;
     }
 
-    private void setupTodayPieChart() {
-        mTodayPieChart.setBackgroundColor(BACKGROUND_COLOR);
-        mTodayPieChart.getDescription().setTextColor(DESCRIPTION_COLOR);
-        mTodayPieChart.setEntryLabelColor(DESCRIPTION_COLOR);
-        mTodayPieChart.getDescription().setText("");
-        mTodayPieChart.setNoDataText("");
-        mTodayPieChart.setTouchEnabled(false);
-        mTodayPieChart.setCenterTextColor(GBApplication.getTextColor(requireContext()));
-        mTodayPieChart.setCenterTextSize(18f);
-        mTodayPieChart.setHoleColor(requireContext().getResources().getColor(R.color.transparent));
-        mTodayPieChart.setHoleRadius(85);
-        mTodayPieChart.setDrawEntryLabels(false);
-        mTodayPieChart.getLegend().setEnabled(false);
-    }
-
     private void setupWeekChart() {
         mWeekChart.setBackgroundColor(BACKGROUND_COLOR);
         mWeekChart.getDescription().setTextColor(DESCRIPTION_COLOR);
@@ -235,8 +219,25 @@ public class PaiChartFragment extends AbstractChartFragment<PaiChartFragment.Pai
     @Override
     protected void updateChartsnUIThread(final PaiChartsData pcd) {
         setupLegend(mWeekChart);
-        mTodayPieChart.setCenterText("");
-        mTodayPieChart.setData(pcd.getDayData().getData());
+        int[] colors = new int[] {
+                ContextCompat.getColor(GBApplication.getContext(), R.color.chart_pai_weekly),
+                ContextCompat.getColor(GBApplication.getContext(), R.color.chart_pai_today)
+        };
+        final int width = (int) TypedValue.applyDimension(
+                TypedValue.COMPLEX_UNIT_DIP,
+                300,
+                GBApplication.getContext().getResources().getDisplayMetrics()
+        );
+        mGoalMinutesGauge.setImageBitmap(GaugeDrawer.drawCircleGaugeSegmented(
+                width,
+                width / 15,
+                colors,
+                pcd.getDayData().getGaugeSegments(),
+                false,
+                String.valueOf(pcd.getDayData().getTotal()),
+                String.valueOf(getPaiTarget()),
+                getContext()
+        ));
 
         // set custom renderer for 30days bar charts
         if (GBApplication.getPrefs().getBoolean("charts_range", true)) {
@@ -261,7 +262,6 @@ public class PaiChartFragment extends AbstractChartFragment<PaiChartFragment.Pai
     @Override
     protected void renderCharts() {
         mWeekChart.invalidate();
-        mTodayPieChart.invalidate();
     }
 
     protected String getWeeksChartsLabel(final Calendar day) {
@@ -321,30 +321,6 @@ public class PaiChartFragment extends AbstractChartFragment<PaiChartFragment.Pai
 
         barChart.getAxisLeft().setAxisMaximum(Math.max(maxPai, getPaiTarget()));
 
-        //LimitLine target = new LimitLine(mTargetValue);
-        //barChart.getAxisLeft().removeAllLimitLines();
-        //barChart.getAxisLeft().addLimitLine(target);
-
-        //float average = 0;
-        //if (TOTAL_DAYS_FOR_AVERAGE > 0) {
-        //    average = Math.abs(balance / TOTAL_DAYS_FOR_AVERAGE);
-        //}
-        //LimitLine average_line = new LimitLine(average);
-        //average_line.setLabel(getString(R.string.average, getAverage(average)));
-//
-        //if (average > (mTargetValue)) {
-        //    average_line.setLineColor(Color.GREEN);
-        //    average_line.setTextColor(Color.GREEN);
-        //} else {
-        //    average_line.setLineColor(Color.RED);
-        //    average_line.setTextColor(Color.RED);
-        //}
-        //if (average > 0) {
-        //    if (GBApplication.getPrefs().getBoolean("charts_show_average", true)) {
-        //        barChart.getAxisLeft().addLimitLine(average_line);
-        //    }
-        //}
-
         return new WeekChartsData(barData, new PreformattedXIndexLabelFormatter(labels));
     }
 
@@ -383,22 +359,14 @@ public class PaiChartFragment extends AbstractChartFragment<PaiChartFragment.Pai
             minutesHigh = 0;
         }
 
-        final PieData data = new PieData();
-        final List<PieEntry> entries = new ArrayList<>();
-
         final int maxPai = Math.max(getPaiTarget(), total);
-        final String todayLabel = today != 0 ? requireContext().getString(R.string.pai_plus_num, today) : "";
 
-        entries.add(new PieEntry(total - today, ""));
-        entries.add(new PieEntry(today, todayLabel));
-        entries.add(new PieEntry(maxPai - total, ""));
+        float[] segments = new float[] {
+                (float) (total - today) / maxPai,
+                (float) today / maxPai
+        };
 
-        final PieDataSet pieDataSet = new PieDataSet(entries, "");
-        pieDataSet.setColors(PAI_TOTAL_COLOR, PAI_DAY_COLOR, 0x0);
-        data.setDataSet(pieDataSet);
-        data.setDrawValues(false);
-
-        return new DayData(day, data, today, total, paiLow, paiModerate, paiHigh, minutesLow, minutesModerate, minutesHigh);
+        return new DayData(day, segments, today, total, paiLow, paiModerate, paiHigh, minutesLow, minutesModerate, minutesHigh);
     }
 
     protected ValueFormatter getRoundFormatter() {
@@ -441,7 +409,7 @@ public class PaiChartFragment extends AbstractChartFragment<PaiChartFragment.Pai
 
     protected static class DayData {
         private final Calendar day;
-        private final PieData data;
+        private final float[] gaugeSegments;
         private final int today;
         private final int total;
         private final int paiLow;
@@ -452,7 +420,7 @@ public class PaiChartFragment extends AbstractChartFragment<PaiChartFragment.Pai
         private final int minutesHigh;
 
         DayData(final Calendar day,
-                final PieData data,
+                float[] gaugeSegments,
                 final int today,
                 final int total,
                 final int paiLow,
@@ -462,7 +430,7 @@ public class PaiChartFragment extends AbstractChartFragment<PaiChartFragment.Pai
                 final int minutesModerate,
                 final int minutesHigh) {
             this.day = day;
-            this.data = data;
+            this.gaugeSegments = gaugeSegments;
             this.today = today;
             this.total = total;
             this.paiLow = paiLow;
@@ -473,8 +441,12 @@ public class PaiChartFragment extends AbstractChartFragment<PaiChartFragment.Pai
             this.minutesHigh = minutesHigh;
         }
 
-        public PieData getData() {
-            return data;
+        public int getTotal() {
+            return total;
+        }
+
+        public float[] getGaugeSegments() {
+            return gaugeSegments;
         }
     }
 
diff --git a/app/src/main/res/layout/fragment_pai_chart.xml b/app/src/main/res/layout/fragment_pai_chart.xml
index 86a716336..7b55cce85 100644
--- a/app/src/main/res/layout/fragment_pai_chart.xml
+++ b/app/src/main/res/layout/fragment_pai_chart.xml
@@ -19,13 +19,15 @@
             android:gravity="center"
             android:textSize="20sp" />
 
-        <com.github.mikephil.charting.charts.PieChart
-            android:id="@+id/pai_chart_today"
-            android:layout_width="200dp"
-            android:layout_height="200dp"
-            android:layout_gravity="bottom|center"
-            android:layout_marginBottom="0dp"
-            android:layout_weight="1" />
+        <ImageView
+            android:layout_weight="2"
+            android:layout_marginTop="15dp"
+            android:layout_marginBottom="10dp"
+            android:id="@+id/goal_minutes_gauge"
+            android:layout_width="180dp"
+            android:layout_height="180dp"
+            android:layout_gravity="center"
+            android:scaleType="fitStart" />
 
         <GridLayout
             android:layout_width="match_parent"
@@ -41,7 +43,7 @@
                     android:layout_width="fill_parent"
                     android:layout_height="5dp"
                     android:layout_marginBottom="10dp"
-                    android:background="@color/chart_activity_dark" />
+                    android:background="@color/chart_pai_today" />
 
                 <TextView
                     android:id="@+id/pai_line_today"
@@ -65,7 +67,7 @@
                     android:layout_width="fill_parent"
                     android:layout_height="5dp"
                     android:layout_marginBottom="10dp"
-                    android:background="@color/chart_deep_sleep_light" />
+                    android:background="@color/chart_pai_weekly" />
 
                 <TextView
                     android:id="@+id/pai_line_total"
@@ -77,7 +79,7 @@
                 <TextView
                     android:layout_width="wrap_content"
                     android:layout_height="wrap_content"
-                    android:text="@string/step_streak_total"
+                    android:text="@string/weekly_total"
                     android:textSize="12sp" />
             </LinearLayout>
 
diff --git a/app/src/main/res/values/colors.xml b/app/src/main/res/values/colors.xml
index f9b4f6ce8..9d630812b 100644
--- a/app/src/main/res/values/colors.xml
+++ b/app/src/main/res/values/colors.xml
@@ -37,6 +37,9 @@
     <color name="chart_activity_light" type="color">#60bd6d</color>
     <color name="chart_activity_dark" type="color">#59b22c</color>
 
+    <color name="chart_pai_weekly" type="color">#007BFF</color>
+    <color name="chart_pai_today" type="color">#2ECC71</color>
+
     <color name="chart_not_worn_light" type="color">#545254</color>
     <color name="chart_not_worn_dark" type="color">#d8d9d8</color>
 
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 5e6188395..7a0750df9 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -2993,6 +2993,7 @@
     <string name="steps_streaks_total_steps_average_hint">Total average %d steps per day</string>
     <string name="step_streaks_achievements_sharing_message">My daily step achievements!</string>
     <string name="step_streaks_achievements_sharing_title">Steps Achievements</string>
+    <string name="weekly_total">Weekly total</string>
     <string name="prefs_hourly_chime">Hourly chime</string>
     <string name="prefs_hourly_chime_summary">The watch will beep once an hour</string>
     <string name="devicetype_flipper_zero">Flipper zero</string>