For #1092: Fixes ordering of custom tab menu items (#2072)

nightly-build-test
Sawyer Blatz 5 years ago committed by GitHub
parent 11918d45f4
commit 4cd0d464a7
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -91,17 +91,17 @@ class CustomTabToolbarMenu(
private val menuItems by lazy {
listOf(
SimpleBrowserMenuItem(
{
val appName = context.getString(R.string.app_name)
context.getString(R.string.browser_menu_powered_by, appName).toUpperCase()
}(),
ToolbarMenu.CAPTION_TEXT_SIZE,
DefaultThemeManager.resolveAttribute(R.attr.primaryText, context)
),
menuToolbar,
BrowserMenuDivider(),
BrowserMenuImageText(
context.getString(R.string.browser_menu_share),
R.drawable.mozac_ic_share
) {
onItemTapped.invoke(ToolbarMenu.Item.Share)
},
SimpleBrowserMenuItem(
{
val appName = context.getString(R.string.app_name)
@ -115,14 +115,16 @@ class CustomTabToolbarMenu(
onItemTapped.invoke(ToolbarMenu.Item.OpenInFenix)
},
BrowserMenuImageText(
context.getString(R.string.browser_menu_share),
R.drawable.mozac_ic_share,
BrowserMenuDivider(),
SimpleBrowserMenuItem(
{
val appName = context.getString(R.string.app_name)
context.getString(R.string.browser_menu_powered_by, appName).toUpperCase()
}(),
ToolbarMenu.CAPTION_TEXT_SIZE,
DefaultThemeManager.resolveAttribute(R.attr.primaryText, context)
) {
onItemTapped.invoke(ToolbarMenu.Item.Share)
},
menuToolbar
)
)
}
}

@ -34,6 +34,7 @@ class CustomTabsIntegration(
toolbar,
sessionId,
customTabToolbarMenu.menuBuilder,
START_OF_MENU_ITEMS_INDEX,
closeListener = { activity?.finish() })
override fun start() {
@ -47,4 +48,8 @@ class CustomTabsIntegration(
override fun onBackPressed(): Boolean {
return feature.onBackPressed()
}
companion object {
const val START_OF_MENU_ITEMS_INDEX = 2
}
}

Loading…
Cancel
Save