From d3030570efd04e861baa5e404c3a31de3e06ae46 Mon Sep 17 00:00:00 2001
From: brahmkshatriya <69040506+brahmkshatriya@users.noreply.github.com>
Date: Wed, 28 Feb 2024 22:06:29 +0530
Subject: [PATCH] Change to tracks in album

---
 .../echo/ui/adapters/PlaylistAdapter.kt       | 11 ++-
 .../echo/ui/album/AlbumFragment.kt            | 23 ++++--
 app/src/main/res/layout/item_track_small.xml  | 82 +++++++++++++++++++
 app/src/main/res/values/strings.xml           |  1 +
 4 files changed, 110 insertions(+), 7 deletions(-)
 create mode 100644 app/src/main/res/layout/item_track_small.xml

diff --git a/app/src/main/java/dev/brahmkshatriya/echo/ui/adapters/PlaylistAdapter.kt b/app/src/main/java/dev/brahmkshatriya/echo/ui/adapters/PlaylistAdapter.kt
index 582e5526..47421ad6 100644
--- a/app/src/main/java/dev/brahmkshatriya/echo/ui/adapters/PlaylistAdapter.kt
+++ b/app/src/main/java/dev/brahmkshatriya/echo/ui/adapters/PlaylistAdapter.kt
@@ -10,6 +10,7 @@ import dev.brahmkshatriya.echo.R
 import dev.brahmkshatriya.echo.common.models.Track
 import dev.brahmkshatriya.echo.databinding.ItemPlaylistItemBinding
 import dev.brahmkshatriya.echo.player.Global
+import dev.brahmkshatriya.echo.player.PlayerHelper.Companion.toTimeString
 import dev.brahmkshatriya.echo.utils.loadInto
 
 class PlaylistAdapter(
@@ -52,8 +53,16 @@ class PlaylistAdapter(
         val track = list[position].second
         binding.playlistItemTitle.text = track.title
         track.cover.loadInto(binding.playlistItemImageView, R.drawable.art_music)
-        binding.playlistItemAuthor.text = track.artists.joinToString(", ") { it.name }
         binding.playlistCurrentItem.isVisible = position == currentPosition
+        var subtitle = ""
+        track.duration?.toTimeString()?.let {
+            subtitle += it
+        }
+        track.artists.joinToString(", ") { it.name }.let {
+            if (it.isNotBlank()) subtitle += if (subtitle.isNotBlank()) " • $it" else it
+        }
+        binding.playlistItemAuthor.isVisible = subtitle.isNotEmpty()
+        binding.playlistItemAuthor.text = subtitle
     }
 
     private fun submitList(list: List<Pair<String, Track>>) {
diff --git a/app/src/main/java/dev/brahmkshatriya/echo/ui/album/AlbumFragment.kt b/app/src/main/java/dev/brahmkshatriya/echo/ui/album/AlbumFragment.kt
index dbf46009..43586684 100644
--- a/app/src/main/java/dev/brahmkshatriya/echo/ui/album/AlbumFragment.kt
+++ b/app/src/main/java/dev/brahmkshatriya/echo/ui/album/AlbumFragment.kt
@@ -22,7 +22,8 @@ import dev.brahmkshatriya.echo.common.models.Album
 import dev.brahmkshatriya.echo.common.models.EchoMediaItem.Companion.toMediaItem
 import dev.brahmkshatriya.echo.common.models.Track
 import dev.brahmkshatriya.echo.databinding.FragmentCollapsingBarBinding
-import dev.brahmkshatriya.echo.databinding.ItemTrackBinding
+import dev.brahmkshatriya.echo.databinding.ItemTrackSmallBinding
+import dev.brahmkshatriya.echo.player.PlayerHelper.Companion.toTimeString
 import dev.brahmkshatriya.echo.ui.MediaItemClickListener
 import dev.brahmkshatriya.echo.ui.adapters.MediaItemsContainerAdapter
 import dev.brahmkshatriya.echo.ui.extension.ExtensionViewModel
@@ -98,7 +99,8 @@ class TrackAdapter(
 
     var list: List<Track>? = null
 
-    inner class ViewHolder(val binding: ItemTrackBinding) : RecyclerView.ViewHolder(binding.root) {
+    inner class ViewHolder(val binding: ItemTrackSmallBinding) :
+        RecyclerView.ViewHolder(binding.root) {
         init {
             binding.root.setOnClickListener {
                 val track = list?.get(bindingAdapterPosition) ?: return@setOnClickListener
@@ -108,7 +110,7 @@ class TrackAdapter(
     }
 
     override fun onCreateViewHolder(parent: ViewGroup, viewType: Int) = ViewHolder(
-        ItemTrackBinding.inflate(LayoutInflater.from(parent.context), parent, false)
+        ItemTrackSmallBinding.inflate(LayoutInflater.from(parent.context), parent, false)
     )
 
     override fun getItemCount() = list?.count() ?: 0
@@ -116,10 +118,19 @@ class TrackAdapter(
     override fun onBindViewHolder(holder: ViewHolder, position: Int) {
         val binding = holder.binding
         val track = list?.get(position) ?: return
-        binding.title.text = track.title
+        binding.itemNumber.text =
+            binding.root.context.getString(R.string.number_dot, (position + 1))
+        binding.itemTitle.text = track.title
         track.cover.loadInto(binding.imageView, R.drawable.art_music)
-        binding.artist.text = track.artists.joinToString(", ") { it.name }
-        binding.album.isVisible = albumVisible
+        var subtitle = ""
+        track.duration?.toTimeString()?.let {
+            subtitle += it
+        }
+        track.artists.joinToString(", ") { it.name }.let {
+            if (it.isNotBlank()) subtitle += if (subtitle.isNotBlank()) " • $it" else it
+        }
+        binding.itemSubtitle.isVisible = subtitle.isNotEmpty()
+        binding.itemSubtitle.text = subtitle
     }
 
     @SuppressLint("NotifyDataSetChanged")
diff --git a/app/src/main/res/layout/item_track_small.xml b/app/src/main/res/layout/item_track_small.xml
new file mode 100644
index 00000000..794b0822
--- /dev/null
+++ b/app/src/main/res/layout/item_track_small.xml
@@ -0,0 +1,82 @@
+<?xml version="1.0" encoding="utf-8"?>
+<FrameLayout 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:layout_width="match_parent"
+    android:layout_height="80dp">
+
+    <TextView
+        android:id="@+id/itemNumber"
+        android:layout_width="52dp"
+        android:gravity="center"
+        android:layout_height="match_parent"
+        android:padding="8dp"
+        android:textStyle="bold"
+        tools:text="1" />
+
+    <LinearLayout
+        android:layout_width="match_parent"
+        android:layout_height="match_parent"
+        android:orientation="horizontal"
+        android:paddingTop="8dp"
+        android:paddingStart="36dp"
+        android:paddingEnd="0dp"
+        android:layout_gravity="center"
+        android:paddingBottom="8dp">
+
+        <com.google.android.material.imageview.ShapeableImageView
+            android:id="@+id/imageView"
+            android:layout_width="64dp"
+            android:layout_height="64dp"
+            android:layout_marginStart="16dp"
+            android:background="?attr/colorSurfaceContainerHigh"
+            android:scaleType="fitCenter"
+            app:shapeAppearance="@style/ShapeAppearance.Material3.Corner.ExtraSmall" />
+
+
+        <LinearLayout
+            android:layout_width="match_parent"
+            android:layout_height="64dp"
+            android:baselineAligned="false"
+            android:gravity="center"
+            android:orientation="horizontal">
+
+            <LinearLayout
+                android:layout_width="0dp"
+                android:layout_height="match_parent"
+                android:layout_marginStart="24dp"
+                android:layout_weight="1"
+                android:baselineAligned="false"
+                android:gravity="center"
+                android:orientation="vertical">
+
+                <TextView
+                    android:id="@+id/itemTitle"
+                    android:layout_width="match_parent"
+                    android:layout_height="wrap_content"
+                    android:maxLines="1"
+                    android:textStyle="bold"
+                    tools:text="Nice Track Title" />
+
+                <TextView
+                    android:id="@+id/itemSubtitle"
+                    android:layout_width="match_parent"
+                    android:layout_height="wrap_content"
+                    android:maxLines="1"
+                    tools:text="Author" />
+            </LinearLayout>
+
+            <com.google.android.material.button.MaterialButton
+                android:id="@+id/itemClose"
+                style="?attr/materialIconButtonStyle"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:layout_margin="8dp"
+                android:contentDescription="@string/next"
+                android:padding="8dp"
+                app:icon="@drawable/ic_more_vert"
+                app:iconSize="24dp"
+                app:iconTint="@color/button_player" />
+        </LinearLayout>
+    </LinearLayout>
+</FrameLayout>
\ No newline at end of file
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index cd0a3a7d..626a6c85 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -28,5 +28,6 @@
     <string name="playlist">Playlist</string>
     <string name="loading">Loading…</string>
     <string name="album">Album</string>
+    <string name="number_dot">%1$d.</string>
 
 </resources>
\ No newline at end of file