Merge pull request #684 from Bnyro/master

minor UI changes

app/src/main/java/com/github/libretube/preferences/AboutFragment.kt

@@ -93,8 +93,14 @@ class AboutFragment : Fragment() {
     private fun showSnackBar(text: String) {
         val snackBar = Snackbar
             .make(binding.root, text, Snackbar.LENGTH_LONG)
+
+        // set snackBar color
         snackBar.setBackgroundTint(getThemeColor(requireContext(), R.attr.colorSurface))
         snackBar.setTextColor(getThemeColor(requireContext(), R.attr.colorPrimary))
+
+        // prevent the text from being partially hidden
+        snackBar.setTextMaxLines(3)
+
         snackBar.show()
     }
 

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

@@ -8,12 +8,16 @@
     <LinearLayout
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
-        android:orientation="vertical">
+        android:layout_marginTop="10dp"
+        android:orientation="vertical"
+        android:paddingStart="20dp"
+        android:paddingEnd="20dp"
+        app:layout_constraintTop_toTopOf="parent">
 
         <LinearLayout
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            android:paddingBottom="16dp"
+            android:layout_marginBottom="16dp"
             app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintTop_toTopOf="parent">
 
@@ -106,8 +110,7 @@
             android:id="@+id/replies_recView"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            android:layout_marginLeft="16dp"
+            android:background="@null"
-            android:layout_marginBottom="10dp"
             android:nestedScrollingEnabled="false" />
 
     </LinearLayout>

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

@@ -322,9 +322,8 @@
                     android:layout_width="match_parent"
                     android:layout_height="wrap_content"
                     android:layout_below="@id/comments_toggle"
-                    android:layout_marginLeft="20dp"
+                    android:layout_marginTop="10dp"
-                    android:layout_marginTop="20dp"
+                    android:layout_marginBottom="10dp"
-                    android:layout_marginRight="20dp"
                     android:nestedScrollingEnabled="false"
                     android:visibility="gone" />
 

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

@@ -2,8 +2,7 @@
 <androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:app="http://schemas.android.com/apk/res-auto"
     android:layout_width="match_parent"
-    android:layout_height="wrap_content"
+    android:layout_height="wrap_content" >
-    android:background="?android:attr/selectableItemBackground">
 
     <LinearLayout
         android:layout_width="match_parent"

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

@@ -1,5 +1,4 @@
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8"?><!-- Replaces the drawables of the default exoplayer controls (see https://github.com/google/ExoPlayer/blob/release-v2/library/ui/src/main/res/values/drawables.xml) -->
-<!-- Replaces the drawables of the default exoplayer controls (see https://github.com/google/ExoPlayer/blob/release-v2/library/ui/src/main/res/values/drawables.xml) -->
 <resources>
 
     <drawable name="exo_styled_controls_play">@drawable/ic_play</drawable>