From 84c91021d138cd0d3b6e4abf69c2f7968cdf3be2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jos=C3=A9=20Rebelo?= <joserebelo@outlook.com>
Date: Thu, 3 Oct 2024 23:29:51 +0100
Subject: [PATCH] Fix body energy page scroll

---
 .../activities/charts/BodyEnergyFragment.java |  7 ++++
 .../main/res/layout/fragment_body_energy.xml  | 39 ++++++++++---------
 2 files changed, 27 insertions(+), 19 deletions(-)

diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/charts/BodyEnergyFragment.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/charts/BodyEnergyFragment.java
index 6839bcf3b..b7f311116 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/charts/BodyEnergyFragment.java
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/charts/BodyEnergyFragment.java
@@ -4,6 +4,7 @@ import android.graphics.Bitmap;
 import android.graphics.Canvas;
 import android.graphics.Color;
 import android.graphics.Paint;
+import android.os.Build;
 import android.os.Bundle;
 import android.util.TypedValue;
 import android.view.LayoutInflater;
@@ -63,6 +64,12 @@ public class BodyEnergyFragment extends AbstractChartFragment<BodyEnergyFragment
     public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
         View rootView = inflater.inflate(R.layout.fragment_body_energy, container, false);
 
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.M) {
+            rootView.setOnScrollChangeListener((v, scrollX, scrollY, oldScrollX, oldScrollY) -> {
+                getChartsHost().enableSwipeRefresh(scrollY == 0);
+            });
+        }
+
         mDateView = rootView.findViewById(R.id.body_energy_date_view);
         bodyEnergyGauge = rootView.findViewById(R.id.body_energy_gauge);
         bodyEnergyGained = rootView.findViewById(R.id.body_energy_gained);
diff --git a/app/src/main/res/layout/fragment_body_energy.xml b/app/src/main/res/layout/fragment_body_energy.xml
index bc7723ec6..8b6d8053f 100644
--- a/app/src/main/res/layout/fragment_body_energy.xml
+++ b/app/src/main/res/layout/fragment_body_energy.xml
@@ -1,4 +1,4 @@
-<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
+<ScrollView xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
@@ -14,51 +14,52 @@
             android:id="@+id/body_energy_date_view"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            android:gravity="center"
-            android:textSize="20sp"
             android:layout_marginTop="15dp"
             android:layout_marginBottom="20dp"
-            />
+            android:gravity="center"
+            android:textSize="20sp" />
 
         <ImageView
+            android:id="@+id/body_energy_gauge"
             android:layout_width="180dp"
             android:layout_height="180dp"
-            android:layout_gravity="center"
             android:layout_centerHorizontal="true"
-            android:scaleType="fitStart"
-            android:id="@+id/body_energy_gauge" />
+            android:layout_gravity="center"
+            android:scaleType="fitStart" />
 
 
         <GridLayout
-            android:background="@color/gauge_line_color"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            android:columnCount="2"
             android:layout_marginTop="15dp"
             android:layout_marginBottom="15dp"
-            >
+            android:background="@color/gauge_line_color"
+            android:columnCount="2">
+
             <LinearLayout
-                android:layout_marginTop="2dp"
-                android:layout_marginEnd="1dp"
                 style="@style/GridTile"
-                >
+                android:layout_marginTop="2dp"
+                android:layout_marginEnd="1dp">
+
                 <TextView
                     android:id="@+id/body_energy_gained"
                     android:layout_width="wrap_content"
                     android:layout_height="wrap_content"
                     android:text="+ 0"
                     android:textSize="20sp" />
+
                 <TextView
                     android:layout_width="wrap_content"
                     android:layout_height="wrap_content"
                     android:text="@string/body_energy_gained"
                     android:textSize="12sp" />
             </LinearLayout>
+
             <LinearLayout
-                android:layout_marginTop="2dp"
-                android:layout_marginEnd="1dp"
                 style="@style/GridTile"
-                >
+                android:layout_marginTop="2dp"
+                android:layout_marginEnd="1dp">
+
                 <TextView
                     android:id="@+id/body_energy_lost"
                     android:layout_width="wrap_content"
@@ -77,8 +78,8 @@
 
         <LinearLayout
             android:layout_width="match_parent"
-            android:layout_height="250sp"
-            >
+            android:layout_height="250sp">
+
             <com.github.mikephil.charting.charts.LineChart
                 android:id="@+id/body_energy_chart"
                 android:layout_width="fill_parent"
@@ -88,4 +89,4 @@
 
     </LinearLayout>
 
-</RelativeLayout>
+</ScrollView>