Draff/TachiyomiSY-Plus
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Replace adapterPosition with bindingAdapterPosition in MigrationProcessHolder

Browse Source
This commit is contained in:
Jobobby04 2020-05-11 15:58:26 -04:00
parent f497e605b4
commit 2b215524b6
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
app/src/main/java/eu/kanade/tachiyomi/ui/migration/manga/process/MigrationProcessHolder.kt
View File

@ -43,7 +43,7 @@ class MigrationProcessHolder(
// correctly positioned. The reason being that the view may change position before the // correctly positioned. The reason being that the view may change position before the
// PopupMenu is shown. // PopupMenu is shown.
migration_menu.setOnClickListener { it.post { showPopupMenu(it) } } migration_menu.setOnClickListener { it.post { showPopupMenu(it) } }
skip_manga.setOnClickListener { it.post { adapter.removeManga(adapterPosition) } } skip_manga.setOnClickListener { it.post { adapter.removeManga(bindingAdapterPosition) } }
} }
fun bind(item: MigrationProcessItem) { fun bind(item: MigrationProcessItem) {
@ -187,7 +187,7 @@ class MigrationProcessHolder(
} }
private fun showPopupMenu(view: View) { private fun showPopupMenu(view: View) {
val item = adapter.getItem(adapterPosition) ?: return val item = adapter.getItem(bindingAdapterPosition) ?: return
// Create a PopupMenu, giving it the clicked view for an anchor // Create a PopupMenu, giving it the clicked view for an anchor
val popup = PopupMenu(view.context, view) val popup = PopupMenu(view.context, view)
@ -206,7 +206,7 @@ class MigrationProcessHolder(
// Set a listener so we are notified if a menu item is clicked // Set a listener so we are notified if a menu item is clicked
popup.setOnMenuItemClickListener { menuItem -> popup.setOnMenuItemClickListener { menuItem ->
adapter.menuItemListener.onMenuItemClick(adapterPosition, menuItem) adapter.menuItemListener.onMenuItemClick(bindingAdapterPosition, menuItem)
true true
} }