diff --git a/mobile/android/android-components/components/browser/toolbar/src/main/java/mozilla/components/browser/toolbar/BrowserToolbar.kt b/mobile/android/android-components/components/browser/toolbar/src/main/java/mozilla/components/browser/toolbar/BrowserToolbar.kt index d5d88457389b..595791f2c579 100644 --- a/mobile/android/android-components/components/browser/toolbar/src/main/java/mozilla/components/browser/toolbar/BrowserToolbar.kt +++ b/mobile/android/android-components/components/browser/toolbar/src/main/java/mozilla/components/browser/toolbar/BrowserToolbar.kt @@ -384,6 +384,7 @@ class BrowserToolbar @JvmOverloads constructor( * @param background A custom (stateful) background drawable resource to be used. * @param padding a custom [Padding] for this Button. * @param iconTintColorResource Optional ID of color resource to tint the icon. + * @param longClickListener Callback that will be invoked whenever the button is long-pressed. * @param listener Callback that will be invoked whenever the button is pressed */ @Suppress("LongParameterList") @@ -460,6 +461,7 @@ class BrowserToolbar @JvmOverloads constructor( * @param longClickListener Callback that will be invoked whenever the button is long-pressed. * @param listener Callback that will be invoked whenever the button is pressed. */ + @Suppress("LongParameterList") open class TwoStateButton( val primaryImage: Drawable, val primaryContentDescription: String, @@ -475,9 +477,9 @@ class BrowserToolbar @JvmOverloads constructor( ) : BrowserToolbar.Button( primaryImage, primaryContentDescription, - longClickListener=longClickListener, + background = background, + longClickListener = longClickListener, listener = listener, - background = background ) { var enabled: Boolean = false private set diff --git a/mobile/android/android-components/components/concept/toolbar/src/main/java/mozilla/components/concept/toolbar/Toolbar.kt b/mobile/android/android-components/components/concept/toolbar/src/main/java/mozilla/components/concept/toolbar/Toolbar.kt index 7cdabe6d10f6..2889e218993e 100644 --- a/mobile/android/android-components/components/concept/toolbar/src/main/java/mozilla/components/concept/toolbar/Toolbar.kt +++ b/mobile/android/android-components/components/concept/toolbar/src/main/java/mozilla/components/concept/toolbar/Toolbar.kt @@ -232,6 +232,7 @@ interface Toolbar { * @param visible Lambda that returns true or false to indicate whether this button should be shown. * @param padding A optional custom padding. * @param iconTintColorResource Optional ID of color resource to tint the icon. + * @param longClickListener Callback that will be invoked whenever the button is long-pressed. * @param listener Callback that will be invoked whenever the button is pressed */ @Suppress("LongParameterList") diff --git a/mobile/android/android-components/samples/toolbar/src/main/java/org/mozilla/samples/toolbar/ToolbarActivity.kt b/mobile/android/android-components/samples/toolbar/src/main/java/org/mozilla/samples/toolbar/ToolbarActivity.kt index 2bd308bfd7fc..f5e193c78903 100644 --- a/mobile/android/android-components/samples/toolbar/src/main/java/org/mozilla/samples/toolbar/ToolbarActivity.kt +++ b/mobile/android/android-components/samples/toolbar/src/main/java/org/mozilla/samples/toolbar/ToolbarActivity.kt @@ -146,7 +146,7 @@ class ToolbarActivity : AppCompatActivity() { // ////////////////////////////////////////////////////////////////////////////////////////// val reload = BrowserToolbar.TwoStateButton( - primaryImage =resources.getThemedDrawable(mozilla.components.ui.icons.R.drawable.mozac_ic_refresh)!!, + primaryImage = resources.getThemedDrawable(mozilla.components.ui.icons.R.drawable.mozac_ic_refresh)!!, primaryContentDescription = "Reload", secondaryImage = resources.getThemedDrawable(mozilla.components.ui.icons.R.drawable.mozac_ic_stop)!!, secondaryContentDescription = "Stop",