diff --git a/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt b/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt index d8e0cd116..cdb536ce9 100644 --- a/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt @@ -532,7 +532,7 @@ class HomeFragment : Fragment(), AccountObserver { val context = requireContext() val components = context.components - action.collection.tabs.forEach { + action.collection.tabs.reversed().forEach { val session = it.restore( context = context, engine = components.core.engine, diff --git a/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/CollectionViewHolder.kt b/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/CollectionViewHolder.kt index da39dfca8..293774726 100644 --- a/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/CollectionViewHolder.kt +++ b/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/CollectionViewHolder.kt @@ -132,25 +132,28 @@ class CollectionItemMenu( private val menuItems by lazy { listOf( SimpleBrowserMenuItem( - context.getString(R.string.collection_delete), - textColorResource = ThemeManager.resolveAttribute(R.attr.destructive, context) + context.getString(R.string.collection_open_tabs) ) { - onItemTapped.invoke(Item.DeleteCollection) + onItemTapped.invoke(Item.OpenTabs) }, - SimpleBrowserMenuItem( - context.getString(R.string.add_tab) - ) { - onItemTapped.invoke(Item.AddTab) - }.apply { visible = { sessionHasOpenTabs } }, + SimpleBrowserMenuItem( context.getString(R.string.collection_rename) ) { onItemTapped.invoke(Item.RenameCollection) }, + SimpleBrowserMenuItem( - context.getString(R.string.collection_open_tabs) + context.getString(R.string.add_tab) ) { - onItemTapped.invoke(Item.OpenTabs) + onItemTapped.invoke(Item.AddTab) + }.apply { visible = { sessionHasOpenTabs } }, + + SimpleBrowserMenuItem( + context.getString(R.string.collection_delete), + textColorResource = ThemeManager.resolveAttribute(R.attr.destructive, context) + ) { + onItemTapped.invoke(Item.DeleteCollection) } ) }