From 681d6cf8a130d4ad5f5c3c47927037b2ead43283 Mon Sep 17 00:00:00 2001 From: David Walsh Date: Tue, 3 Mar 2020 11:05:41 -0600 Subject: [PATCH] For #8652 - Allow sharing multiple bookmarks --- .../mozilla/fenix/library/bookmarks/BookmarkFragment.kt | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkFragment.kt b/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkFragment.kt index 9cacf8e59..8cdaa9323 100644 --- a/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkFragment.kt @@ -166,8 +166,6 @@ class BookmarkFragment : LibraryPageFragment(), UserInteractionHan } else { inflater.inflate(R.menu.bookmarks_select_multi, menu) } - - menu.findItem(R.id.share_bookmark_multi_select)?.isVisible = mode.selectedItems.size == 1 } } } @@ -196,10 +194,12 @@ class BookmarkFragment : LibraryPageFragment(), UserInteractionHan true } R.id.share_bookmark_multi_select -> { - val bookmark = bookmarkStore.state.mode.selectedItems.first() + val shareTabs = bookmarkStore.state.mode.selectedItems.map { + ShareData(url = it.url, title = it.title) + } navigate( BookmarkFragmentDirections.actionBookmarkFragmentToShareFragment( - data = arrayOf(ShareData(url = bookmark.url, title = bookmark.title)) + data = shareTabs.toTypedArray() ) ) true