For #12551 - Implement swipe to delete for bookmarks.
parent
313e5abf6a
commit
cd2532349d
|
@ -4,16 +4,16 @@
|
||||||
|
|
||||||
package org.mozilla.fenix.library.bookmarks
|
package org.mozilla.fenix.library.bookmarks
|
||||||
|
|
||||||
|
import android.view.LayoutInflater
|
||||||
import android.view.View
|
import android.view.View
|
||||||
import android.view.ViewGroup
|
import android.view.ViewGroup
|
||||||
import android.view.ViewGroup.LayoutParams.MATCH_PARENT
|
|
||||||
import android.view.ViewGroup.LayoutParams.WRAP_CONTENT
|
|
||||||
import androidx.core.view.isVisible
|
import androidx.core.view.isVisible
|
||||||
import androidx.recyclerview.widget.DiffUtil
|
import androidx.recyclerview.widget.DiffUtil
|
||||||
import androidx.recyclerview.widget.RecyclerView
|
import androidx.recyclerview.widget.RecyclerView
|
||||||
import mozilla.appservices.places.BookmarkRoot
|
import mozilla.appservices.places.BookmarkRoot
|
||||||
import mozilla.components.concept.storage.BookmarkNode
|
import mozilla.components.concept.storage.BookmarkNode
|
||||||
import mozilla.components.concept.storage.BookmarkNodeType
|
import mozilla.components.concept.storage.BookmarkNodeType
|
||||||
|
import org.mozilla.fenix.R
|
||||||
import org.mozilla.fenix.library.LibrarySiteItemView
|
import org.mozilla.fenix.library.LibrarySiteItemView
|
||||||
import org.mozilla.fenix.library.SelectionHolder
|
import org.mozilla.fenix.library.SelectionHolder
|
||||||
import org.mozilla.fenix.library.bookmarks.viewholders.BookmarkFolderViewHolder
|
import org.mozilla.fenix.library.bookmarks.viewholders.BookmarkFolderViewHolder
|
||||||
|
@ -68,9 +68,8 @@ class BookmarkAdapter(val emptyView: View, val interactor: BookmarkViewInteracto
|
||||||
}
|
}
|
||||||
|
|
||||||
override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): BookmarkNodeViewHolder {
|
override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): BookmarkNodeViewHolder {
|
||||||
val view = LibrarySiteItemView(parent.context).apply {
|
val view = LayoutInflater.from(parent.context)
|
||||||
layoutParams = ViewGroup.LayoutParams(MATCH_PARENT, WRAP_CONTENT)
|
.inflate(R.layout.bookmark_list_item, parent, false) as LibrarySiteItemView
|
||||||
}
|
|
||||||
|
|
||||||
return when (viewType) {
|
return when (viewType) {
|
||||||
LibrarySiteItemView.ItemType.SITE.ordinal -> BookmarkItemViewHolder(view, interactor, this)
|
LibrarySiteItemView.ItemType.SITE.ordinal -> BookmarkItemViewHolder(view, interactor, this)
|
||||||
|
|
|
@ -0,0 +1,124 @@
|
||||||
|
/* This Source Code Form is subject to the terms of the Mozilla Public
|
||||||
|
* License, v. 2.0. If a copy of the MPL was not distributed with this
|
||||||
|
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */
|
||||||
|
|
||||||
|
package org.mozilla.fenix.library.bookmarks
|
||||||
|
|
||||||
|
import android.graphics.Canvas
|
||||||
|
import android.graphics.Rect
|
||||||
|
import android.graphics.drawable.Drawable
|
||||||
|
import androidx.appcompat.content.res.AppCompatResources
|
||||||
|
import androidx.recyclerview.widget.ItemTouchHelper
|
||||||
|
import androidx.recyclerview.widget.RecyclerView
|
||||||
|
import mozilla.components.support.ktx.android.content.getColorFromAttr
|
||||||
|
import mozilla.components.support.ktx.android.content.getDrawableWithTint
|
||||||
|
import mozilla.components.support.ktx.android.util.dpToPx
|
||||||
|
import org.mozilla.fenix.R
|
||||||
|
import org.mozilla.fenix.home.sessioncontrol.SwipeToDeleteCallback
|
||||||
|
import org.mozilla.fenix.library.bookmarks.viewholders.BookmarkNodeViewHolder
|
||||||
|
import org.mozilla.fenix.library.bookmarks.viewholders.BookmarkSeparatorViewHolder
|
||||||
|
|
||||||
|
class BookmarkTouchHelper(interactor: BookmarkViewInteractor) :
|
||||||
|
ItemTouchHelper(BookmarkTouchCallback(interactor))
|
||||||
|
|
||||||
|
class BookmarkTouchCallback(private val interactor: BookmarkViewInteractor) :
|
||||||
|
ItemTouchHelper.SimpleCallback(0, ItemTouchHelper.LEFT or ItemTouchHelper.RIGHT) {
|
||||||
|
|
||||||
|
override fun getSwipeDirs(
|
||||||
|
recyclerView: RecyclerView,
|
||||||
|
viewHolder: RecyclerView.ViewHolder
|
||||||
|
): Int {
|
||||||
|
val item = (viewHolder as BookmarkNodeViewHolder).item
|
||||||
|
return if (viewHolder is BookmarkSeparatorViewHolder || item?.inRoots() == true) {
|
||||||
|
0
|
||||||
|
} else {
|
||||||
|
super.getSwipeDirs(recyclerView, viewHolder)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Delete the bookmark when swiped.
|
||||||
|
*/
|
||||||
|
override fun onSwiped(viewHolder: RecyclerView.ViewHolder, direction: Int) {
|
||||||
|
val item = (viewHolder as BookmarkNodeViewHolder).item
|
||||||
|
item?.let { interactor.onDelete(setOf(it)) }
|
||||||
|
}
|
||||||
|
|
||||||
|
override fun onChildDraw(
|
||||||
|
c: Canvas,
|
||||||
|
recyclerView: RecyclerView,
|
||||||
|
viewHolder: RecyclerView.ViewHolder,
|
||||||
|
dX: Float,
|
||||||
|
dY: Float,
|
||||||
|
actionState: Int,
|
||||||
|
isCurrentlyActive: Boolean
|
||||||
|
) {
|
||||||
|
super.onChildDraw(c, recyclerView, viewHolder, dX, dY, actionState, isCurrentlyActive)
|
||||||
|
val icon = recyclerView.context.getDrawableWithTint(
|
||||||
|
R.drawable.ic_delete,
|
||||||
|
recyclerView.context.getColorFromAttr(R.attr.destructive)
|
||||||
|
)!!
|
||||||
|
val background = AppCompatResources.getDrawable(
|
||||||
|
recyclerView.context,
|
||||||
|
R.drawable.swipe_delete_background
|
||||||
|
)!!
|
||||||
|
val margin =
|
||||||
|
SwipeToDeleteCallback.MARGIN.dpToPx(recyclerView.context.resources.displayMetrics)
|
||||||
|
val cellHeight = viewHolder.itemView.bottom - viewHolder.itemView.top
|
||||||
|
val iconTop = viewHolder.itemView.top + (cellHeight - icon.intrinsicHeight) / 2
|
||||||
|
val iconBottom = iconTop + icon.intrinsicHeight
|
||||||
|
|
||||||
|
when {
|
||||||
|
dX > 0 -> { // Swiping to the right
|
||||||
|
val backgroundBounds = Rect(
|
||||||
|
viewHolder.itemView.left, viewHolder.itemView.top,
|
||||||
|
(viewHolder.itemView.left + dX).toInt() + SwipeToDeleteCallback.BACKGROUND_CORNER_OFFSET,
|
||||||
|
viewHolder.itemView.bottom
|
||||||
|
)
|
||||||
|
val iconLeft = viewHolder.itemView.left + margin
|
||||||
|
val iconRight = viewHolder.itemView.left + margin + icon.intrinsicWidth
|
||||||
|
val iconBounds = Rect(iconLeft, iconTop, iconRight, iconBottom)
|
||||||
|
|
||||||
|
setBounds(background, backgroundBounds, icon, iconBounds)
|
||||||
|
draw(background, icon, c)
|
||||||
|
}
|
||||||
|
dX < 0 -> { // Swiping to the left
|
||||||
|
val backgroundBounds = Rect(
|
||||||
|
(viewHolder.itemView.right + dX).toInt() - SwipeToDeleteCallback.BACKGROUND_CORNER_OFFSET,
|
||||||
|
viewHolder.itemView.top, viewHolder.itemView.right, viewHolder.itemView.bottom
|
||||||
|
)
|
||||||
|
val iconLeft = viewHolder.itemView.right - margin - icon.intrinsicWidth
|
||||||
|
val iconRight = viewHolder.itemView.right - margin
|
||||||
|
val iconBounds = Rect(iconLeft, iconTop, iconRight, iconBottom)
|
||||||
|
|
||||||
|
setBounds(background, backgroundBounds, icon, iconBounds)
|
||||||
|
draw(background, icon, c)
|
||||||
|
}
|
||||||
|
else -> { // View not swiped
|
||||||
|
val bounds = Rect(0, 0, 0, 0)
|
||||||
|
setBounds(background, bounds, icon, bounds)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
override fun onMove(
|
||||||
|
recyclerView: RecyclerView,
|
||||||
|
viewHolder: RecyclerView.ViewHolder,
|
||||||
|
target: RecyclerView.ViewHolder
|
||||||
|
): Boolean = false
|
||||||
|
|
||||||
|
private fun setBounds(
|
||||||
|
background: Drawable,
|
||||||
|
backgroundBounds: Rect,
|
||||||
|
icon: Drawable,
|
||||||
|
iconBounds: Rect
|
||||||
|
) {
|
||||||
|
background.bounds = backgroundBounds
|
||||||
|
icon.bounds = iconBounds
|
||||||
|
}
|
||||||
|
|
||||||
|
private fun draw(background: Drawable, icon: Drawable, c: Canvas) {
|
||||||
|
background.draw(c)
|
||||||
|
icon.draw(c)
|
||||||
|
}
|
||||||
|
}
|
|
@ -13,8 +13,8 @@ import kotlinx.android.synthetic.main.component_bookmark.view.*
|
||||||
import mozilla.appservices.places.BookmarkRoot
|
import mozilla.appservices.places.BookmarkRoot
|
||||||
import mozilla.components.concept.storage.BookmarkNode
|
import mozilla.components.concept.storage.BookmarkNode
|
||||||
import mozilla.components.support.base.feature.UserInteractionHandler
|
import mozilla.components.support.base.feature.UserInteractionHandler
|
||||||
import org.mozilla.fenix.R
|
|
||||||
import org.mozilla.fenix.NavGraphDirections
|
import org.mozilla.fenix.NavGraphDirections
|
||||||
|
import org.mozilla.fenix.R
|
||||||
import org.mozilla.fenix.library.LibraryPageView
|
import org.mozilla.fenix.library.LibraryPageView
|
||||||
import org.mozilla.fenix.library.SelectionInteractor
|
import org.mozilla.fenix.library.SelectionInteractor
|
||||||
|
|
||||||
|
@ -125,6 +125,8 @@ class BookmarkView(
|
||||||
view.swipe_refresh.setOnRefreshListener {
|
view.swipe_refresh.setOnRefreshListener {
|
||||||
interactor.onRequestSync()
|
interactor.onRequestSync()
|
||||||
}
|
}
|
||||||
|
|
||||||
|
BookmarkTouchHelper(interactor).attachToRecyclerView(view.bookmark_list)
|
||||||
}
|
}
|
||||||
|
|
||||||
fun update(state: BookmarkFragmentState) {
|
fun update(state: BookmarkFragmentState) {
|
||||||
|
|
|
@ -25,7 +25,10 @@ class BookmarkFolderViewHolder(
|
||||||
private val selectionHolder: SelectionHolder<BookmarkNode>
|
private val selectionHolder: SelectionHolder<BookmarkNode>
|
||||||
) : BookmarkNodeViewHolder(view, interactor) {
|
) : BookmarkNodeViewHolder(view, interactor) {
|
||||||
|
|
||||||
|
override var item: BookmarkNode? = null
|
||||||
|
|
||||||
override fun bind(item: BookmarkNode) {
|
override fun bind(item: BookmarkNode) {
|
||||||
|
this.item = item
|
||||||
|
|
||||||
containerView.displayAs(LibrarySiteItemView.ItemType.FOLDER)
|
containerView.displayAs(LibrarySiteItemView.ItemType.FOLDER)
|
||||||
|
|
||||||
|
|
|
@ -20,7 +20,10 @@ class BookmarkItemViewHolder(
|
||||||
private val selectionHolder: SelectionHolder<BookmarkNode>
|
private val selectionHolder: SelectionHolder<BookmarkNode>
|
||||||
) : BookmarkNodeViewHolder(view, interactor) {
|
) : BookmarkNodeViewHolder(view, interactor) {
|
||||||
|
|
||||||
|
override var item: BookmarkNode? = null
|
||||||
|
|
||||||
override fun bind(item: BookmarkNode) {
|
override fun bind(item: BookmarkNode) {
|
||||||
|
this.item = item
|
||||||
|
|
||||||
containerView.displayAs(LibrarySiteItemView.ItemType.SITE)
|
containerView.displayAs(LibrarySiteItemView.ItemType.SITE)
|
||||||
|
|
||||||
|
|
|
@ -18,8 +18,9 @@ import org.mozilla.fenix.library.bookmarks.BookmarkViewInteractor
|
||||||
abstract class BookmarkNodeViewHolder(
|
abstract class BookmarkNodeViewHolder(
|
||||||
override val containerView: LibrarySiteItemView,
|
override val containerView: LibrarySiteItemView,
|
||||||
private val interactor: BookmarkViewInteractor
|
private val interactor: BookmarkViewInteractor
|
||||||
) :
|
) : RecyclerView.ViewHolder(containerView), LayoutContainer {
|
||||||
RecyclerView.ViewHolder(containerView), LayoutContainer {
|
|
||||||
|
abstract var item: BookmarkNode?
|
||||||
|
|
||||||
abstract fun bind(item: BookmarkNode)
|
abstract fun bind(item: BookmarkNode)
|
||||||
|
|
||||||
|
|
|
@ -16,7 +16,10 @@ class BookmarkSeparatorViewHolder(
|
||||||
interactor: BookmarkViewInteractor
|
interactor: BookmarkViewInteractor
|
||||||
) : BookmarkNodeViewHolder(view, interactor) {
|
) : BookmarkNodeViewHolder(view, interactor) {
|
||||||
|
|
||||||
|
override var item: BookmarkNode? = null
|
||||||
|
|
||||||
override fun bind(item: BookmarkNode) {
|
override fun bind(item: BookmarkNode) {
|
||||||
|
this.item = item
|
||||||
containerView.displayAs(LibrarySiteItemView.ItemType.SEPARATOR)
|
containerView.displayAs(LibrarySiteItemView.ItemType.SEPARATOR)
|
||||||
setupMenu(item)
|
setupMenu(item)
|
||||||
}
|
}
|
||||||
|
|
|
@ -0,0 +1,8 @@
|
||||||
|
<?xml version="1.0" encoding="utf-8"?>
|
||||||
|
<!-- This Source Code Form is subject to the terms of the Mozilla Public
|
||||||
|
- License, v. 2.0. If a copy of the MPL was not distributed with this
|
||||||
|
- file, You can obtain one at http://mozilla.org/MPL/2.0/. -->
|
||||||
|
<org.mozilla.fenix.library.LibrarySiteItemView xmlns:android="http://schemas.android.com/apk/res/android"
|
||||||
|
android:layout_width="match_parent"
|
||||||
|
android:layout_height="wrap_content"
|
||||||
|
android:background="?foundation" />
|
Loading…
Reference in New Issue