From 851f066a2ac775fc349132d12e681cd38b3ee50b Mon Sep 17 00:00:00 2001 From: Emily Kager Date: Tue, 21 May 2019 14:43:51 -0700 Subject: [PATCH] For #2377 - Added ability to share multiple history items --- .../org/mozilla/fenix/library/history/HistoryFragment.kt | 8 ++++++-- 1 file changed, 6 insertions(+), 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 index 15f93195e..27ea339c3 100644 --- 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 }