For mozilla-mobile#8652 - Allow sharing multiple bookmarks

darkwing · Mar 3, 2020 · e272010 · e272010
1 parent e6e2dd9
commit e272010
Showing 1 changed file with 4 additions and 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
@@ -165,8 +165,6 @@ class BookmarkFragment : LibraryPageFragment<BookmarkNode>(), UserInteractionHan
                 } else {
                     inflater.inflate(R.menu.bookmarks_select_multi, menu)
                 }
-
-                menu.findItem(R.id.share_bookmark_multi_select)?.isVisible = mode.selectedItems.size == 1
             }
         }
     }
@@ -195,10 +193,12 @@ class BookmarkFragment : LibraryPageFragment<BookmarkNode>(), 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