Bug 1812712 - implement open in background in bookmark manager

mozilla-mobile · Mar 22, 2024 · aa17830 · aa17830
1 parent 98ad1b0
commit aa17830
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/fenix/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkController.kt b/fenix/app/src/main/java/org/mozilla/fenix/library/bookmarks/BookmarkController.kt
@@ -48,6 +48,12 @@ interface BookmarkController {
     fun handleAllBookmarksDeselected()
     fun handleCopyUrl(item: BookmarkNode)
     fun handleBookmarkSharing(item: BookmarkNode)
+    /**
+     * Handles opening bookmark
+     * @param item Bookmark node.
+     * @param mode Browsing mode.
+     * @param openInBackground If bookmark should open in background or not
+     */
     fun handleOpeningBookmark(item: BookmarkNode, mode: BrowsingMode, openInBackground: Boolean)
     fun handleOpeningFolderBookmarks(folder: BookmarkNode, mode: BrowsingMode)
 
@@ -166,8 +172,9 @@ class DefaultBookmarkController(
     ) {
         openInNewTab(item.url!!, mode)
         showSnackbar(resources.getString(R.string.bookmark_opened_in_background))
-        if (!openInBackground)
+        if (!openInBackground) {
             showTabTray(mode.isPrivate)
+        }
     }
 
     private fun extractURLsFromTree(node: BookmarkNode): MutableList<String> {