From 29d5c5f76a3926e64ad95c99260fb2c3353f810b Mon Sep 17 00:00:00 2001
From: Andries Smit <mr.andries.smit@gmail.com>
Date: Mon, 31 Jan 2022 13:20:00 +0100
Subject: [PATCH] Improve statuslight alligning and spacing

---
 .../layout/overview_statuslights_layout.xml   | 76 +++++++++----------
 1 file changed, 34 insertions(+), 42 deletions(-)

diff --git a/app/src/main/res/layout/overview_statuslights_layout.xml b/app/src/main/res/layout/overview_statuslights_layout.xml
index 53794cfb61..0738364e34 100644
--- a/app/src/main/res/layout/overview_statuslights_layout.xml
+++ b/app/src/main/res/layout/overview_statuslights_layout.xml
@@ -1,6 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:tools="http://schemas.android.com/tools"
     android:id="@+id/status_lights"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
@@ -15,14 +16,13 @@
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:layout_weight="1"
-        android:focusable="true">
+        android:focusable="true"
+        android:gravity="center_horizontal">
 
         <ImageView
-            android:id="@+id/cannula_or_patch"
-            android:layout_width="wrap_content"
+            android:layout_width="38dp"
             android:layout_height="fill_parent"
             android:contentDescription="@string/statuslights_cannula_age"
-            android:scaleType="centerInside"
             android:scaleX="2"
             android:scaleY="2"
             app:srcCompat="@drawable/ic_cp_age_cannula" />
@@ -31,9 +31,8 @@
             android:id="@+id/cannula_age"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:paddingStart="1dp"
-            android:paddingEnd="2dp"
-            android:textSize="14sp" />
+            android:lines="1"
+            tools:text="1d12h" />
 
     </LinearLayout>
 
@@ -41,59 +40,55 @@
         android:layout_width="wrap_content"
         android:layout_height="fill_parent"
         android:layout_weight="1"
-        android:focusable="true">
+        android:focusable="true"
+        android:gravity="center_horizontal">
 
         <ImageView
-            android:layout_width="wrap_content"
+            android:layout_width="26dp"
             android:layout_height="fill_parent"
             android:contentDescription="@string/a11y_insulin_label"
-            android:gravity="center_vertical"
-            android:scaleType="centerInside"
-            android:scaleX="1.8"
-            android:scaleY="1.8"
+            android:scaleX="1.7"
+            android:scaleY="1.7"
             app:srcCompat="@drawable/ic_cp_age_insulin" />
 
         <TextView
             android:id="@+id/insulin_age"
             android:layout_width="wrap_content"
             android:layout_height="fill_parent"
-            android:gravity="center_vertical"
-            android:paddingStart="1dp"
-            android:paddingEnd="2dp"
-            android:textSize="14sp" />
+            android:lines="1"
+            tools:text="12h" />
 
         <TextView
             android:id="@+id/reservoir_level"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:gravity="center_vertical"
-            android:textSize="14sp" />
+            android:lines="1"
+            android:paddingStart="2dp"
+            tools:text="50+U" />
+
     </LinearLayout>
 
     <LinearLayout
         android:layout_width="wrap_content"
         android:layout_height="fill_parent"
         android:layout_weight="1"
-        android:focusable="true">
+        android:focusable="true"
+        android:gravity="center_horizontal">
 
         <ImageView
-            android:layout_width="wrap_content"
+            android:layout_width="28dp"
             android:layout_height="fill_parent"
             android:contentDescription="@string/careportal_sensor_label"
-            android:gravity="center_vertical"
-            android:scaleType="centerInside"
-            android:scaleX="2"
-            android:scaleY="2"
+            android:scaleX="1.6"
+            android:scaleY="1.6"
             app:srcCompat="@drawable/ic_cp_age_sensor" />
 
         <TextView
             android:id="@+id/sensor_age"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:gravity="center_vertical"
-            android:paddingStart="1dp"
-            android:paddingEnd="2dp"
-            android:textSize="14sp" />
+            android:lines="1"
+            tools:text="1d12h" />
 
     </LinearLayout>
 
@@ -101,34 +96,31 @@
         android:layout_width="wrap_content"
         android:layout_height="fill_parent"
         android:layout_weight="1"
-        android:focusable="true">
+        android:focusable="true"
+        android:gravity="center_horizontal">
 
         <ImageView
-            android:layout_width="wrap_content"
+            android:layout_width="32dp"
             android:layout_height="wrap_content"
             android:contentDescription="@string/battery_label"
-            android:scaleType="centerInside"
-            android:scaleX="2"
-            android:scaleY="2"
+            android:scaleX="1.8"
+            android:scaleY="1.8"
             app:srcCompat="@drawable/ic_cp_age_battery" />
 
         <TextView
             android:id="@+id/pb_age"
             android:layout_width="wrap_content"
             android:layout_height="fill_parent"
-            android:gravity="center_vertical"
-            android:paddingStart="1dp"
-            android:paddingEnd="1dp"
-            android:textSize="14sp" />
+            android:lines="1"
+            tools:text="-" />
 
         <TextView
             android:id="@+id/battery_level"
             android:layout_width="wrap_content"
             android:layout_height="fill_parent"
-            android:gravity="center_vertical"
-            android:paddingStart="1dp"
-            android:paddingEnd="2dp"
-            android:textSize="14sp" />
+            android:lines="1"
+            android:paddingStart="2dp"
+            tools:text="100%" />
 
     </LinearLayout>