Skip to content

Commit 79d2266

Browse files
authored
Add new items to the Tab Switcher's overflow menu (#343)
1 parent dc26134 commit 79d2266

File tree

2 files changed

+17
-1
lines changed

2 files changed

+17
-1
lines changed

app/src/main/java/com/duckduckgo/app/tabs/ui/TabSwitcherActivity.kt

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ import com.duckduckgo.app.browser.R
2727
import com.duckduckgo.app.global.DuckDuckGoActivity
2828
import com.duckduckgo.app.global.view.ClearPersonalDataAction
2929
import com.duckduckgo.app.global.view.FireDialog
30+
import com.duckduckgo.app.settings.SettingsActivity
3031
import com.duckduckgo.app.statistics.pixels.Pixel
3132
import com.duckduckgo.app.tabs.model.TabEntity
3233
import com.duckduckgo.app.tabs.ui.TabSwitcherViewModel.Command
@@ -98,8 +99,9 @@ class TabSwitcherActivity : DuckDuckGoActivity(), TabSwitcherAdapter.TabSwitched
9899
override fun onOptionsItemSelected(item: MenuItem): Boolean {
99100
when (item.itemId) {
100101
R.id.fire -> onFire()
101-
R.id.newTab -> onNewTabRequested()
102+
R.id.newTab, R.id.newTabOverflow -> onNewTabRequested()
102103
R.id.closeAllTabs -> closeAllTabs()
104+
R.id.settings -> showSettings()
103105
}
104106
return super.onOptionsItemSelected(item)
105107
}
@@ -129,6 +131,10 @@ class TabSwitcherActivity : DuckDuckGoActivity(), TabSwitcherAdapter.TabSwitched
129131
}
130132
}
131133

134+
private fun showSettings() {
135+
startActivity(SettingsActivity.intent(this))
136+
}
137+
132138
override fun finish() {
133139
clearObserversEarlyToStopViewUpdates()
134140
super.finish()

app/src/main/res/menu/menu_tab_switcher_activity.xml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,9 +31,19 @@
3131
android:title="@string/newTabMenuItem"
3232
app:showAsAction="always" />
3333

34+
<item
35+
android:id="@+id/newTabOverflow"
36+
android:title="@string/newTabMenuItem"
37+
app:showAsAction="never" />
38+
3439
<item
3540
android:id="@+id/closeAllTabs"
3641
android:title="@string/closeAllTabsMenuItem"
3742
app:showAsAction="never" />
3843

44+
<item
45+
android:id="@+id/settings"
46+
android:title="@string/settingsMenuItemTitle"
47+
app:showAsAction="never" />
48+
3949
</menu>

0 commit comments

Comments
 (0)