fix demo transition bug

zhangkun0018 · Nov 27, 2021 · 361a615 · 361a615
1 parent e6c6343
commit 361a615
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/app/src/main/java/com/github/iielse/imageviewer/demo/business/TestDataAdapter.kt b/app/src/main/java/com/github/iielse/imageviewer/demo/business/TestDataAdapter.kt
@@ -3,6 +3,7 @@ package com.github.iielse.imageviewer.demo.business
 import android.view.ViewGroup
 import androidx.recyclerview.widget.RecyclerView
 import com.github.iielse.imageviewer.demo.core.BasePagedAdapter
+import com.github.iielse.imageviewer.demo.core.viewer.ViewerTransitionHelper
 import com.github.iielse.imageviewer.demo.data.MyData
 
 class TestDataAdapter : BasePagedAdapter() {
@@ -19,4 +20,12 @@ class TestDataAdapter : BasePagedAdapter() {
             is TestDataViewHolder -> item?.data<MyData>()?.let { holder.bind(it, position) }
         }
     }
+
+    override fun onViewAttachedToWindow(holder: RecyclerView.ViewHolder) {
+        super.onViewAttachedToWindow(holder)
+        if (holder is TestDataViewHolder) {
+            val photoId = (holder.itemView.tag as? MyData?)?.id ?: return
+            ViewerTransitionHelper.put(photoId, holder.binding.imageView)
+        }
+    }
 }
diff --git a/app/src/main/java/com/github/iielse/imageviewer/demo/business/TestDataViewHolder.kt b/app/src/main/java/com/github/iielse/imageviewer/demo/business/TestDataViewHolder.kt
@@ -38,7 +38,7 @@ class TestDataViewHolder(
         binding.imageView.scaleType = if (pos == 19) ImageView.ScaleType.FIT_XY else ImageView.ScaleType.CENTER_CROP
 
         Glide.with(binding.imageView).load(item.url).into(binding.imageView)
-        ViewerTransitionHelper.put(item.id, binding.imageView)
+
     }
 }