From e18a08f88a118695d7020bc7a268211938bb18cd Mon Sep 17 00:00:00 2001 From: iorgamgabriel Date: Wed, 6 Mar 2024 13:52:42 +0200 Subject: [PATCH] Bug 1883418 - Translate icon is should be visible on Private Browsing Mode --- .../main/java/org/mozilla/fenix/browser/BrowserFragment.kt | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt b/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt index bb0f710bb..089c65dbb 100644 --- a/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/browser/BrowserFragment.kt @@ -323,10 +323,9 @@ class BrowserFragment : BaseBrowserFragment(), UserInteractionHandler { AppCompatResources.getDrawable( context, R.drawable.mozac_ic_translate_24, - )!!.apply { - setTint(ContextCompat.getColor(context, R.color.fx_mobile_text_color_primary)) - }, + ), contentDescription = context.getString(R.string.browser_toolbar_translate), + iconTintColorResource = ThemeManager.resolveAttribute(R.attr.textPrimary, context), visible = { translationsAvailable }, listener = { browserToolbarInteractor.onTranslationsButtonClicked() @@ -346,7 +345,7 @@ class BrowserFragment : BaseBrowserFragment(), UserInteractionHandler { tintColorResource = if (isTranslated) { R.color.fx_mobile_icon_color_accent_violet } else { - R.color.fx_mobile_text_color_primary + ThemeManager.resolveAttribute(R.attr.textPrimary, context) }, contentDescription = if (isTranslated) { context.getString(