detail: add framework for disc number items

Add the UI and data components for a disc number value within the album
detail view.

This is the first-step of a multi-step addition to finally implement
disc numbers.
This commit is contained in:
OxygenCobalt 2022-05-18 17:30:50 -06:00
parent 5381e9f9a2
commit 6f2a0d66c6
No known key found for this signature in database
GPG key ID: 37DBE3621FE9AD47
5 changed files with 69 additions and 5 deletions

View file

@ -46,9 +46,11 @@ object IntegerTable {
const val ITEM_TYPE_GENRE_DETAIL = 0xA00B const val ITEM_TYPE_GENRE_DETAIL = 0xA00B
/** GenreSongViewHolder */ /** GenreSongViewHolder */
const val ITEM_TYPE_GENRE_SONG = 0xA00C const val ITEM_TYPE_GENRE_SONG = 0xA00C
/** DiscHeaderViewHolder */
const val ITEM_TYPE_DISC_HEADER = 0xA00D
/** QueueSongViewHolder */ /** QueueSongViewHolder */
const val ITEM_TYPE_QUEUE_SONG = 0xA00D const val ITEM_TYPE_QUEUE_SONG = 0xA00E
/** "Music playback" Notification code */ /** "Music playback" Notification code */
const val NOTIFICATION_CODE = 0xA0A0 const val NOTIFICATION_CODE = 0xA0A0

View file

@ -21,6 +21,7 @@ import androidx.lifecycle.LiveData
import androidx.lifecycle.MutableLiveData import androidx.lifecycle.MutableLiveData
import androidx.lifecycle.ViewModel import androidx.lifecycle.ViewModel
import org.oxycblt.auxio.R import org.oxycblt.auxio.R
import org.oxycblt.auxio.detail.recycler.DiscHeader
import org.oxycblt.auxio.detail.recycler.SortHeader import org.oxycblt.auxio.detail.recycler.SortHeader
import org.oxycblt.auxio.music.Album import org.oxycblt.auxio.music.Album
import org.oxycblt.auxio.music.Artist import org.oxycblt.auxio.music.Artist
@ -137,7 +138,7 @@ class DetailViewModel : ViewModel() {
logD("Refreshing album data") logD("Refreshing album data")
val data = mutableListOf<Item>(album) val data = mutableListOf<Item>(album)
data.add(SortHeader(id = -2, R.string.lbl_songs)) data.add(SortHeader(id = -2, R.string.lbl_songs))
data.addAll(albumSort.album(album)) data.add(DiscHeader(id = -3, 1))
_albumData.value = data _albumData.value = data
} }
} }

View file

@ -25,6 +25,7 @@ import org.oxycblt.auxio.R
import org.oxycblt.auxio.coil.bindAlbumCover import org.oxycblt.auxio.coil.bindAlbumCover
import org.oxycblt.auxio.databinding.ItemAlbumSongBinding import org.oxycblt.auxio.databinding.ItemAlbumSongBinding
import org.oxycblt.auxio.databinding.ItemDetailBinding import org.oxycblt.auxio.databinding.ItemDetailBinding
import org.oxycblt.auxio.databinding.ItemDiscHeaderBinding
import org.oxycblt.auxio.music.Album import org.oxycblt.auxio.music.Album
import org.oxycblt.auxio.music.Song import org.oxycblt.auxio.music.Song
import org.oxycblt.auxio.ui.BindingViewHolder import org.oxycblt.auxio.ui.BindingViewHolder
@ -50,6 +51,7 @@ class AlbumDetailAdapter(listener: Listener) :
super.getCreatorFromItem(item) super.getCreatorFromItem(item)
?: when (item) { ?: when (item) {
is Album -> AlbumDetailViewHolder.CREATOR is Album -> AlbumDetailViewHolder.CREATOR
is DiscHeader -> DiscHeaderViewHolder.CREATOR
is Song -> AlbumSongViewHolder.CREATOR is Song -> AlbumSongViewHolder.CREATOR
else -> null else -> null
} }
@ -58,6 +60,7 @@ class AlbumDetailAdapter(listener: Listener) :
super.getCreatorFromViewType(viewType) super.getCreatorFromViewType(viewType)
?: when (viewType) { ?: when (viewType) {
AlbumDetailViewHolder.CREATOR.viewType -> AlbumDetailViewHolder.CREATOR AlbumDetailViewHolder.CREATOR.viewType -> AlbumDetailViewHolder.CREATOR
DiscHeaderViewHolder.CREATOR.viewType -> DiscHeaderViewHolder.CREATOR
AlbumSongViewHolder.CREATOR.viewType -> AlbumSongViewHolder.CREATOR AlbumSongViewHolder.CREATOR.viewType -> AlbumSongViewHolder.CREATOR
else -> null else -> null
} }
@ -67,6 +70,7 @@ class AlbumDetailAdapter(listener: Listener) :
when (item) { when (item) {
is Album -> (viewHolder as AlbumDetailViewHolder).bind(item, listener) is Album -> (viewHolder as AlbumDetailViewHolder).bind(item, listener)
is DiscHeader -> (viewHolder as DiscHeaderViewHolder).bind(item, Unit)
is Song -> (viewHolder as AlbumSongViewHolder).bind(item, listener) is Song -> (viewHolder as AlbumSongViewHolder).bind(item, listener)
} }
} }
@ -100,8 +104,8 @@ class AlbumDetailAdapter(listener: Listener) :
return when { return when {
oldItem is Album && newItem is Album -> oldItem is Album && newItem is Album ->
AlbumDetailViewHolder.DIFFER.areItemsTheSame(oldItem, newItem) AlbumDetailViewHolder.DIFFER.areItemsTheSame(oldItem, newItem)
oldItem is SortHeader && newItem is SortHeader -> oldItem is DiscHeader && newItem is DiscHeader ->
SortHeaderViewHolder.DIFFER.areItemsTheSame(oldItem, newItem) DiscHeaderViewHolder.DIFFER.areItemsTheSame(oldItem, newItem)
oldItem is Song && newItem is Song -> oldItem is Song && newItem is Song ->
AlbumSongViewHolder.DIFFER.areItemsTheSame(oldItem, newItem) AlbumSongViewHolder.DIFFER.areItemsTheSame(oldItem, newItem)
else -> DetailAdapter.DIFFER.areItemsTheSame(oldItem, newItem) else -> DetailAdapter.DIFFER.areItemsTheSame(oldItem, newItem)
@ -162,6 +166,32 @@ private class AlbumDetailViewHolder private constructor(private val binding: Ite
} }
} }
data class DiscHeader(override val id: Long, val disc: Int) : Item()
class DiscHeaderViewHolder(private val binding: ItemDiscHeaderBinding) :
BindingViewHolder<DiscHeader, Unit>(binding.root) {
override fun bind(item: DiscHeader, listener: Unit) {
binding.discNo.textSafe = "Disc 1"
}
companion object {
val CREATOR =
object : Creator<DiscHeaderViewHolder> {
override val viewType: Int
get() = IntegerTable.ITEM_TYPE_DISC_HEADER
override fun create(context: Context) =
DiscHeaderViewHolder(ItemDiscHeaderBinding.inflate(context.inflater))
}
val DIFFER =
object : SimpleItemCallback<DiscHeader>() {
override fun areItemsTheSame(oldItem: DiscHeader, newItem: DiscHeader) =
oldItem.disc == newItem.disc
}
}
}
private class AlbumSongViewHolder private constructor(private val binding: ItemAlbumSongBinding) : private class AlbumSongViewHolder private constructor(private val binding: ItemAlbumSongBinding) :
BindingViewHolder<Song, MenuItemListener>(binding.root), Highlightable { BindingViewHolder<Song, MenuItemListener>(binding.root), Highlightable {
override fun bind(item: Song, listener: MenuItemListener) { override fun bind(item: Song, listener: MenuItemListener) {

View file

@ -224,7 +224,7 @@ class MediaSessionComponent(private val context: Context, private val player: Pl
// Position updates arrive faster when you upload a state that is different, as it // Position updates arrive faster when you upload a state that is different, as it
// forces the system to re-poll the position. // forces the system to re-poll the position.
// FIXME: For some reason however, positions just DON'T UPDATE AT ALL when you // FIXME: For some reason however, positions just DON'T UPDATE AT ALL when you
// change from FROM THE APP ONLY WHEN THE PLAYER IS PAUSED. AAAAAAAAAAAAAAAAAAAAAAAAAA // change from FROM THE APP ONLY WHEN THE PLAYER IS PAUSED.
val state = val state =
PlaybackStateCompat.Builder() PlaybackStateCompat.Builder()
.setActions(ACTIONS) .setActions(ACTIONS)

View file

@ -0,0 +1,31 @@
<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout 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:padding="@dimen/spacing_medium"
android:layout_width="match_parent"
android:layout_height="wrap_content">
<org.oxycblt.auxio.coil.StyledImageView
android:id="@+id/disc_item"
style="@style/Widget.Auxio.Image.Small"
android:layout_width="@dimen/size_cover_compact"
android:scaleType="center"
android:src="@drawable/ic_album"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toTopOf="parent"
tools:ignore="ContentDescription" />
<TextView
android:id="@+id/disc_no"
style="@style/Widget.Auxio.TextView.Header"
android:layout_width="0dp"
android:layout_height="wrap_content"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintStart_toEndOf="@+id/disc_item"
app:layout_constraintTop_toTopOf="parent"
tools:text="Disc 16" />
</androidx.constraintlayout.widget.ConstraintLayout>