From 281567995e46ecb2706f258f2c89b513bbf0a67d Mon Sep 17 00:00:00 2001 From: Emily Kager Date: Thu, 23 May 2019 23:35:05 -0700 Subject: [PATCH] [fenix] For https://github.com/mozilla-mobile/fenix/issues/2815 - Use RootView for displaying snackbar in browserfragment (https://github.com/mozilla-mobile/fenix/pull/2816) --- .../java/org/mozilla/fenix/browser/BrowserFragment.kt | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt b/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt index 1358dda5d..43808b12c 100644 --- a/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt @@ -423,9 +423,11 @@ class BrowserFragment : Fragment(), BackHandler, CoroutineScope { is SearchAction.ToolbarLongClicked -> { getSessionById()?.let { session -> session.copyUrl(requireContext()) - FenixSnackbar.make(view!!, Snackbar.LENGTH_LONG) - .setText(resources.getString(R.string.url_copied)) - .show() + view?.rootView?.let { + FenixSnackbar.make(it, Snackbar.LENGTH_LONG) + .setText(resources.getString(R.string.url_copied)) + .show() + } } } } @@ -507,7 +509,7 @@ class BrowserFragment : Fragment(), BackHandler, CoroutineScope { requireComponents.analytics.metrics.track(Event.AddBookmark) view?.let { FenixSnackbar.make( - it, + it.rootView, Snackbar.LENGTH_LONG ) .setAction(getString(R.string.edit_bookmark_snackbar_action)) {