diff --git a/android-components/components/concept/toolbar/src/test/java/mozilla/components/concept/toolbar/ActionButtonTest.kt b/android-components/components/concept/toolbar/src/test/java/mozilla/components/concept/toolbar/ActionButtonTest.kt index 6b33deac2580..ddcbccdfe97a 100644 --- a/android-components/components/concept/toolbar/src/test/java/mozilla/components/concept/toolbar/ActionButtonTest.kt +++ b/android-components/components/concept/toolbar/src/test/java/mozilla/components/concept/toolbar/ActionButtonTest.kt @@ -45,7 +45,15 @@ class ActionButtonTest { @Test fun `constructor with drawables`() { val visibilityListener = { false } - val button = Toolbar.ActionButton(mock(), "image", visibilityListener, { false }, 0, null) { } + val button = Toolbar.ActionButton( + mock(), + "image", + visibilityListener, + { false }, + { -1 }, + 0, + null, + ) { } assertNotNull(button.imageDrawable) assertEquals("image", button.contentDescription) assertEquals(visibilityListener, button.visible) diff --git a/fenix/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt b/fenix/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt index 442df5c3fa4c..1f08ba179dc9 100644 --- a/fenix/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt +++ b/fenix/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt @@ -303,6 +303,7 @@ class BrowserFragment : BaseBrowserFragment(), UserInteractionHandler { R.drawable.mozac_ic_share_android_24, )!!, contentDescription = getString(R.string.browser_menu_share), + weight = { 4 }, iconTintColorResource = ThemeManager.resolveAttribute(R.attr.textPrimary, context), listener = { browserToolbarInteractor.onShareActionClicked() }, ) @@ -394,6 +395,7 @@ class BrowserFragment : BaseBrowserFragment(), UserInteractionHandler { )!!, secondaryContentDescription = context.getString(R.string.browser_menu_stop), disableInSecondaryState = false, + weight = { 5 }, longClickListener = { browserToolbarInteractor.onBrowserToolbarMenuItemTapped( ToolbarMenu.Item.Reload(bypassCache = true),