From e78e7daf0ada9b3399a97aab4a5eace86e8fc3db Mon Sep 17 00:00:00 2001
From: Bnyro <yannick.fehler@web.de>
Date: Tue, 9 Aug 2022 14:37:40 +0200
Subject: [PATCH] minor player UI improvements

---
 .../github/libretube/fragments/PlayerFragment.kt |  1 -
 app/src/main/res/drawable/ic_pause.xml           |  6 +++---
 .../layout/exo_styled_player_control_view.xml    | 16 +++++++++++-----
 app/src/main/res/values/style.xml                |  4 ++--
 4 files changed, 16 insertions(+), 11 deletions(-)

diff --git a/app/src/main/java/com/github/libretube/fragments/PlayerFragment.kt b/app/src/main/java/com/github/libretube/fragments/PlayerFragment.kt
index 9d09776d2..d29c500e4 100644
--- a/app/src/main/java/com/github/libretube/fragments/PlayerFragment.kt
+++ b/app/src/main/java/com/github/libretube/fragments/PlayerFragment.kt
@@ -82,7 +82,6 @@ import com.google.android.exoplayer2.trackselection.DefaultTrackSelector
 import com.google.android.exoplayer2.ui.AspectRatioFrameLayout
 import com.google.android.exoplayer2.ui.CaptionStyleCompat
 import com.google.android.exoplayer2.ui.StyledPlayerView
-import com.google.android.exoplayer2.ui.TimeBar
 import com.google.android.exoplayer2.upstream.DataSource
 import com.google.android.exoplayer2.upstream.DefaultDataSource
 import com.google.android.exoplayer2.upstream.DefaultHttpDataSource
diff --git a/app/src/main/res/drawable/ic_pause.xml b/app/src/main/res/drawable/ic_pause.xml
index 645e31d2c..b92538943 100644
--- a/app/src/main/res/drawable/ic_pause.xml
+++ b/app/src/main/res/drawable/ic_pause.xml
@@ -4,7 +4,7 @@
     android:tint="?attr/colorControlNormal"
     android:viewportWidth="24"
     android:viewportHeight="24">
-    <path
-        android:fillColor="@android:color/white"
-        android:pathData="M6,19h4L10,5L6,5v14zM14,5v14h4L18,5h-4z" />
+  <path
+      android:pathData="M6.719,19H10.24V5H6.719ZM13.76,5v14h3.521V5Z"
+      android:fillColor="@android:color/white"/>
 </vector>
diff --git a/app/src/main/res/layout/exo_styled_player_control_view.xml b/app/src/main/res/layout/exo_styled_player_control_view.xml
index 94e317f32..4cebb2abc 100644
--- a/app/src/main/res/layout/exo_styled_player_control_view.xml
+++ b/app/src/main/res/layout/exo_styled_player_control_view.xml
@@ -70,7 +70,8 @@
                 <TextView
                     android:id="@+id/speed_text"
                     style="@style/PlayerControlTop"
-                    android:text="1x" />
+                    android:text="1x"
+                    tools:ignore="HardcodedText" />
 
                 <TextView
                     android:id="@+id/quality_text"
@@ -80,7 +81,7 @@
                 <ImageButton
                     android:id="@+id/toggle_options"
                     style="@style/PlayerControlTop"
-                    android:layout_marginTop="-.5dp"
+                    android:layout_marginTop="-1dp"
                     android:src="@drawable/ic_arrow_down"
                     app:tint="@android:color/white" />
 
@@ -165,7 +166,7 @@
             android:layout_height="wrap_content"
             android:baselineAligned="false"
             android:paddingStart="10dp"
-            android:paddingEnd="15dp">
+            android:paddingEnd="20dp">
 
             <LinearLayout
                 android:id="@id/exo_time"
@@ -176,16 +177,20 @@
 
                 <TextView
                     android:id="@id/exo_position"
-                    style="@style/TimeString" />
+                    style="@style/TimeString"
+                    tools:text="05:20" />
 
                 <TextView
                     style="@style/TimeString"
                     android:text="  •  "
+                    android:textColor="#B3FFFFFF"
                     tools:ignore="HardcodedText" />
 
                 <TextView
                     android:id="@id/exo_duration"
-                    style="@style/TimeString" />
+                    style="@style/TimeString"
+                    android:textColor="#B3FFFFFF"
+                    tools:text="12:15" />
 
             </LinearLayout>
 
@@ -264,6 +269,7 @@
             android:layout_height="15dp"
             android:layout_gravity="bottom"
             android:layout_marginLeft="10dp"
+            android:layout_marginTop="5dp"
             android:layout_marginRight="10dp"
             android:layout_marginBottom="10dp">
 
diff --git a/app/src/main/res/values/style.xml b/app/src/main/res/values/style.xml
index d7d85f1d4..911da05bb 100644
--- a/app/src/main/res/values/style.xml
+++ b/app/src/main/res/values/style.xml
@@ -141,8 +141,8 @@
         <item name="android:layout_gravity">center_vertical</item>
         <item name="android:layout_height">wrap_content</item>
         <item name="android:layout_width">wrap_content</item>
-        <item name="paddingStart">4dp</item>
-        <item name="paddingEnd">4dp</item>
+        <item name="paddingStart">5dp</item>
+        <item name="paddingEnd">5dp</item>
         <item name="android:textSize">14sp</item>
         <item name="android:textStyle">bold</item>
         <item name="android:textColor">@android:color/white</item>