From 3d169a733d371d02351ee6327a0d7304aa2497d0 Mon Sep 17 00:00:00 2001 From: mcarare Date: Thu, 9 Jun 2022 15:23:52 +0300 Subject: [PATCH] [fenix] For https://github.com/mozilla-mobile/fenix/issues/25563: Show Snackbar only if fragment is still attached. --- .../historymetadata/HistoryMetadataGroupFragment.kt | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/app/src/main/java/org/mozilla/fenix/library/historymetadata/HistoryMetadataGroupFragment.kt b/app/src/main/java/org/mozilla/fenix/library/historymetadata/HistoryMetadataGroupFragment.kt index e09869334..173a97ea1 100644 --- a/app/src/main/java/org/mozilla/fenix/library/historymetadata/HistoryMetadataGroupFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/library/historymetadata/HistoryMetadataGroupFragment.kt @@ -35,6 +35,7 @@ import org.mozilla.fenix.ext.requireComponents import org.mozilla.fenix.ext.setTextColor import org.mozilla.fenix.ext.showToolbar import org.mozilla.fenix.ext.components +import org.mozilla.fenix.ext.runIfFragmentIsAttached import org.mozilla.fenix.ext.toShortUrl import org.mozilla.fenix.library.LibraryPageFragment import org.mozilla.fenix.library.history.History @@ -231,10 +232,12 @@ class HistoryMetadataGroupFragment : } private fun allDeletedSnackbar() { - showSnackBar( - requireView(), - getString(R.string.delete_history_group_snackbar) - ) + runIfFragmentIsAttached { + showSnackBar( + binding.root, + getString(R.string.delete_history_group_snackbar) + ) + } } private fun showTabTray() {