Improve downloads list

app/src/main/kotlin/org/koitharu/kotatsu/download/ui/list/DownloadItemAD.kt

@@ -1,5 +1,6 @@
 package org.koitharu.kotatsu.download.ui.list
 
+import android.transition.TransitionManager
 import android.view.View
 import androidx.core.content.ContextCompat
 import androidx.core.view.isVisible
@@ -14,12 +15,14 @@ import org.koitharu.kotatsu.core.ui.BaseListAdapter
 import org.koitharu.kotatsu.core.ui.image.TrimTransformation
 import org.koitharu.kotatsu.core.util.ext.drawableEnd
 import org.koitharu.kotatsu.core.util.ext.enqueueWith
+import org.koitharu.kotatsu.core.util.ext.isAnimationsEnabled
 import org.koitharu.kotatsu.core.util.ext.newImageRequest
 import org.koitharu.kotatsu.core.util.ext.source
 import org.koitharu.kotatsu.core.util.ext.textAndVisible
 import org.koitharu.kotatsu.databinding.ItemDownloadBinding
 import org.koitharu.kotatsu.download.ui.list.chapters.DownloadChapter
 import org.koitharu.kotatsu.download.ui.list.chapters.downloadChapterAD
+import org.koitharu.kotatsu.list.ui.ListModelDiffCallback
 import org.koitharu.kotatsu.list.ui.adapter.ListItemType
 import org.koitharu.kotatsu.list.ui.model.ListModel
 import org.koitharu.kotatsu.parsers.util.format
@@ -60,6 +63,9 @@ fun downloadItemAD(
 	binding.recyclerViewChapters.adapter = chaptersAdapter
 
 	bind { payloads ->
+		if (ListModelDiffCallback.PAYLOAD_CHECKED_CHANGED in payloads && context.isAnimationsEnabled) {
+			TransitionManager.beginDelayedTransition(binding.constraintLayout)
+		}
 		binding.textViewTitle.text = item.manga.title
 		binding.imageViewCover.newImageRequest(lifecycleOwner, item.manga.coverUrl)?.apply {
 			placeholder(R.drawable.ic_placeholder)

app/src/main/kotlin/org/koitharu/kotatsu/download/ui/list/DownloadItemModel.kt

@@ -3,6 +3,7 @@ package org.koitharu.kotatsu.download.ui.list
 import android.text.format.DateUtils
 import androidx.work.WorkInfo
 import org.koitharu.kotatsu.download.ui.list.chapters.DownloadChapter
+import org.koitharu.kotatsu.list.ui.ListModelDiffCallback
 import org.koitharu.kotatsu.list.ui.model.ListModel
 import org.koitharu.kotatsu.parsers.model.Manga
 import java.util.Date
@@ -57,17 +58,10 @@ data class DownloadItemModel(
 		return other is DownloadItemModel && other.id == id
 	}
 
-	override fun getChangePayload(previousState: ListModel): Any? {
-		return when (previousState) {
-			is DownloadItemModel -> {
-				if (workState == previousState.workState) {
-					Unit
-				} else {
-					null
-				}
-			}
-
-			else -> super.getChangePayload(previousState)
-		}
+	override fun getChangePayload(previousState: ListModel): Any? = when {
+		previousState !is DownloadItemModel -> super.getChangePayload(previousState)
+		workState != previousState.workState -> null
+		isExpanded != previousState.isExpanded -> ListModelDiffCallback.PAYLOAD_CHECKED_CHANGED
+		else -> ListModelDiffCallback.PAYLOAD_ANYTHING_CHANGED
 	}
 }

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

@@ -8,6 +8,7 @@
 	android:layout_height="wrap_content">
 
 	<androidx.constraintlayout.widget.ConstraintLayout
+		android:id="@+id/constraintLayout"
 		android:layout_width="match_parent"
 		android:layout_height="match_parent"
 		android:paddingBottom="12dp">
@@ -106,10 +107,12 @@
 		<com.google.android.material.card.MaterialCardView
 			android:id="@+id/card_details"
 			android:layout_width="0dp"
-			android:layout_height="wrap_content"
+			android:layout_height="0dp"
 			android:layout_marginHorizontal="12dp"
 			android:layout_marginTop="12dp"
 			app:layout_constraintEnd_toEndOf="parent"
+			app:layout_constraintHeight_default="wrap"
+			app:layout_constraintHeight_max="280dp"
 			app:layout_constraintStart_toStartOf="parent"
 			app:layout_constraintTop_toBottomOf="@id/progressBar"
 			app:shapeAppearance="?shapeAppearanceCornerMedium">
@@ -117,8 +120,12 @@
 			<androidx.recyclerview.widget.RecyclerView
 				android:id="@+id/recyclerView_chapters"
 				android:layout_width="match_parent"
-				android:layout_height="wrap_content"
-				app:layoutManager="androidx.recyclerview.widget.LinearLayoutManager" />
+				android:layout_height="match_parent"
+				android:orientation="vertical"
+				android:scrollbars="vertical"
+				app:layoutManager="androidx.recyclerview.widget.LinearLayoutManager"
+				tools:itemCount="200"
+				tools:listitem="@layout/item_chapter_download" />
 
 		</com.google.android.material.card.MaterialCardView>