ishan
/
fenix
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

For #27366 - Revert "For #26644 - Apply the new theme earlier to the private browsing button" 

This reverts commit c7c5682104 given that the
severity of the new issue is much higher.
This commit is contained in:
Mugurell 2022-10-11 15:37:08 +03:00 committed by mergify[bot]
parent 8b0fd1b85a
commit f0aced49c0
1 changed files with 1 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt
View File

@ -39,7 +39,6 @@ import androidx.navigation.fragment.navArgs
import androidx.recyclerview.widget.LinearLayoutManager
import androidx.recyclerview.widget.LinearSmoothScroller
import androidx.recyclerview.widget.RecyclerView.SmoothScroller
import androidx.vectordrawable.graphics.drawable.VectorDrawableCompat
import com.google.android.material.appbar.AppBarLayout
import com.google.android.material.button.MaterialButton
import com.google.android.material.snackbar.Snackbar
@ -994,22 +993,7 @@ class HomeFragment : Fragment() {
}
binding.wordmarkText.imageTintList = tintColor
// Need to preemptively apply the new theme to the private browsing button drawable
// See https://github.com/mozilla-mobile/fenix/issues/26644#issuecomment-1254961616
(activity as? HomeActivity)?.themeManager?.let { themeManager ->
with(binding.privateBrowsingButton) {
val drawable = VectorDrawableCompat.create(
resources,
R.drawable.private_browsing_button,
resources.newTheme().apply {
applyStyle(themeManager.currentThemeResource, true)
},
)
setImageDrawable(drawable)
imageTintList = tintColor
}
}
binding.privateBrowsingButton.imageTintList = tintColor
}
private fun observeWallpaperUpdates() {