From 4b59fc89397e06cb289c9fa3e22cc1f02f5f191e Mon Sep 17 00:00:00 2001 From: ekager Date: Fri, 19 Apr 2019 14:43:08 -0700 Subject: [PATCH] Closes #1835 - Do not crash because of snackbar --- .../mozilla/fenix/browser/BrowserFragment.kt | 32 ++++++++++--------- 1 file changed, 17 insertions(+), 15 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 a06d1ad0ea..b97648cab0 100644 --- a/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt @@ -358,21 +358,23 @@ class BrowserFragment : Fragment(), BackHandler, CoroutineScope { .addItem(BookmarkRoot.Mobile.id, session.url, session.title, null) launch(Main) { requireComponents.analytics.metrics.track(Event.AddBookmark) - FenixSnackbar.make( - view!!, - Snackbar.LENGTH_LONG - ) - .setAction(getString(R.string.edit_bookmark_snackbar_action)) { - Navigation.findNavController(requireActivity(), R.id.container) - .navigate( - BrowserFragmentDirections - .actionBrowserFragmentToBookmarkEditFragment( - guid - ) - ) - } - .setText(getString(R.string.bookmark_saved_snackbar)) - .show() + view?.let { + FenixSnackbar.make( + it, + Snackbar.LENGTH_LONG + ) + .setAction(getString(R.string.edit_bookmark_snackbar_action)) { + Navigation.findNavController(requireActivity(), R.id.container) + .navigate( + BrowserFragmentDirections + .actionBrowserFragmentToBookmarkEditFragment( + guid + ) + ) + } + .setText(getString(R.string.bookmark_saved_snackbar)) + .show() + } } } }