For #2377 - Added ability to share multiple history items

mozilla-mobile · May 22, 2019 · 851f066 · 851f066
1 parent e5f27a2
commit 851f066
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/app/src/main/java/org/mozilla/fenix/library/history/HistoryFragment.kt b/app/src/main/java/org/mozilla/fenix/library/history/HistoryFragment.kt
@@ -42,7 +42,6 @@ import org.mozilla.fenix.ext.share
 import org.mozilla.fenix.mvi.ActionBusFactory
 import org.mozilla.fenix.mvi.getAutoDisposeObservable
 import org.mozilla.fenix.mvi.getManagedEmitter
-import org.mozilla.fenix.utils.ItsNotBrokenSnack
 import java.net.MalformedURLException
 import java.net.URL
 import kotlin.coroutines.CoroutineContext
@@ -184,7 +183,12 @@ class HistoryFragment : Fragment(), CoroutineScope, BackHandler {
                 val selectedHistory = (historyComponent.uiView as HistoryUIView).getSelected()
                 when {
                     selectedHistory.size == 1 -> context?.share(selectedHistory.first().url)
-                    selectedHistory.size > 1 -> ItsNotBrokenSnack(context!!).showSnackbar(issueNumber = "2377")
+                    selectedHistory.size > 1 -> {
+                        val shareText = selectedHistory.joinToString("\n") {
+                            it.url
+                        }
+                        requireContext().share(shareText)
+                    }
                 }
                 true
             }