From adab132bf3214414426ca1ae13ddca304ab45f5a Mon Sep 17 00:00:00 2001
From: Prathamesh More <prathameshmm02@gmail.com>
Date: Thu, 16 Dec 2021 00:36:28 +0530
Subject: [PATCH] [Home] Consistent no. of lines for Home playlist buttons

---
 .../retromusic/fragments/home/HomeFragment.kt    | 16 ++++++++++++++++
 app/src/main/res/layout/abs_playlists.xml        |  1 -
 app/src/main/res/values/styles.xml               |  3 ++-
 3 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/app/src/main/java/code/name/monkey/retromusic/fragments/home/HomeFragment.kt b/app/src/main/java/code/name/monkey/retromusic/fragments/home/HomeFragment.kt
index 647361a84..3390621d4 100644
--- a/app/src/main/java/code/name/monkey/retromusic/fragments/home/HomeFragment.kt
+++ b/app/src/main/java/code/name/monkey/retromusic/fragments/home/HomeFragment.kt
@@ -23,6 +23,7 @@ import android.view.View
 import androidx.activity.addCallback
 import androidx.core.os.bundleOf
 import androidx.core.text.HtmlCompat
+import androidx.core.view.doOnLayout
 import androidx.core.view.doOnPreDraw
 import androidx.navigation.fragment.FragmentNavigatorExtras
 import androidx.navigation.fragment.findNavController
@@ -87,6 +88,21 @@ class HomeFragment :
             remove()
             mainActivity.finish()
         }
+        view.doOnLayout {
+            adjustPlaylistButtons()
+        }
+    }
+
+    private fun adjustPlaylistButtons() {
+        val buttons =
+            listOf(binding.history, binding.lastAdded, binding.topPlayed, binding.actionShuffle)
+        buttons.maxOf { it.lineCount }.let { maxLineCount->
+            buttons.forEach { button ->
+                // Set the highest line count to every button for consistency
+                button.setLines(maxLineCount)
+            }
+        }
+
     }
 
     private fun setupListeners() {
diff --git a/app/src/main/res/layout/abs_playlists.xml b/app/src/main/res/layout/abs_playlists.xml
index bf4b5e80c..ee11757cd 100644
--- a/app/src/main/res/layout/abs_playlists.xml
+++ b/app/src/main/res/layout/abs_playlists.xml
@@ -43,7 +43,6 @@
         android:layout_marginEnd="16dp"
         android:text="@string/my_top_tracks"
         app:icon="@drawable/ic_trending_up"
-        app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintEnd_toStartOf="@+id/actionShuffle"
         app:layout_constraintHorizontal_bias="0.5"
         app:layout_constraintStart_toStartOf="@+id/history"
diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml
index d1fbc9261..159bd3f86 100644
--- a/app/src/main/res/values/styles.xml
+++ b/app/src/main/res/values/styles.xml
@@ -223,7 +223,8 @@
     </style>
 
     <style name="HomeActionButton" parent="Widget.Material3.Button.ElevatedButton.Icon">
-        <item name="android:padding">16dp</item>
+        <item name="android:paddingTop">16dp</item>
+        <item name="android:paddingBottom">16dp</item>
         <item name="android:textColor">?attr/colorOnSurface</item>
         <item name="shapeAppearanceOverlay">
             @style/ShapeAppearanceOverlay.Material3.FloatingActionButton