fix: gyroscope screen crash and UI (#2651)

Author: rahul31124

app/src/main/java/io/pslab/fragment/GyroscopeDataFragment.java

@@ -425,8 +425,11 @@ private void writeLogToFile(long timestamp, float readingX, float readingY, floa
     }
 
     private void visualizeData() {
+        if (!isAdded()) return;
         for (int i = 0; i < gyroscopeViewFragments.size(); i++) {
             GyroscopeViewFragment fragment = gyroscopeViewFragments.get(i);
+            if (!fragment.isAdded()) continue;
+
             long timeElapsed = (System.currentTimeMillis() - startTime) / 1000;
             if (timeElapsed != fragment.getPreviousTimeElapsed()) {
                 fragment.setPreviousTimeElapsed(timeElapsed);

app/src/main/res/layout/fragment_gyroscope_data.xml

@@ -1,40 +1,41 @@
 <?xml version="1.0" encoding="utf-8"?>
-<LinearLayout android:id="@+id/gyro_linearlayout"
+<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"
-    android:fitsSystemWindows="true"
-    android:orientation="vertical"
-    tools:context="io.pslab.activity.GyroscopeActivity">
+    android:fitsSystemWindows="true">
 
-    <fragment
-        android:id="@+id/gyroscope_x_axis_fragment"
-        android:name="io.pslab.fragment.GyroscopeViewFragment"
+    <LinearLayout
+        android:id="@+id/gyro_linearlayout"
         android:layout_width="match_parent"
-        android:layout_height="0dp"
-        android:layout_weight="1"
-        android:layout_margin="5dp"
-        tools:layout="@layout/gyroscope_list_item" />
+        android:layout_height="wrap_content"
+        android:orientation="vertical"
+        tools:context="io.pslab.activity.GyroscopeActivity">
 
-    <fragment
-        android:id="@+id/gyroscope_y_axis_fragment"
-        android:name="io.pslab.fragment.GyroscopeViewFragment"
-        android:layout_width="match_parent"
-        android:layout_height="0dp"
-        android:layout_weight="1"
-        android:layout_margin="5dp"
-        tools:layout="@layout/gyroscope_list_item" />
+        <fragment
+            android:id="@+id/gyroscope_x_axis_fragment"
+            android:name="io.pslab.fragment.GyroscopeViewFragment"
+            android:layout_width="match_parent"
+            android:layout_height="@dimen/gyroscope_fragment_height"
+            android:layout_margin="@dimen/gyroscope_fragment_margin"
+            tools:layout="@layout/gyroscope_list_item" />
 
-    <fragment
-        android:id="@+id/gyroscope_z_axis_fragment"
-        android:name="io.pslab.fragment.GyroscopeViewFragment"
-        android:layout_width="match_parent"
-        android:layout_height="0dp"
-        android:layout_weight="1"
-        android:layout_marginTop="5dp"
-        android:layout_marginLeft="5dp"
-        android:layout_marginRight="5dp"
-        android:layout_marginBottom="10dp"
-        tools:layout="@layout/gyroscope_list_item" />
-</LinearLayout>
+        <fragment
+            android:id="@+id/gyroscope_y_axis_fragment"
+            android:name="io.pslab.fragment.GyroscopeViewFragment"
+            android:layout_width="match_parent"
+            android:layout_height="@dimen/gyroscope_fragment_height"
+            android:layout_margin="@dimen/gyroscope_fragment_margin"
+            tools:layout="@layout/gyroscope_list_item" />
+
+        <fragment
+            android:id="@+id/gyroscope_z_axis_fragment"
+            android:name="io.pslab.fragment.GyroscopeViewFragment"
+            android:layout_width="match_parent"
+            android:layout_height="@dimen/gyroscope_fragment_height"
+            android:layout_margin="@dimen/gyroscope_fragment_margin"
+            tools:layout="@layout/gyroscope_list_item" />
+
+    </LinearLayout>
+</ScrollView>

app/src/main/res/values/dimens.xml

@@ -421,4 +421,7 @@
     <dimen name="text_size_osc">13sp</dimen>
     <dimen name="text_size_9sp">9sp</dimen>
 
+    <dimen name="gyroscope_fragment_height">300dp</dimen>
+    <dimen name="gyroscope_fragment_margin">5dp</dimen>
+
 </resources>