From 722ab9f3cadaece6aa0a07b18d3f0b6c9dc7c543 Mon Sep 17 00:00:00 2001 From: Jonathan Almeida Date: Wed, 13 Oct 2021 01:22:03 -0400 Subject: [PATCH] Issue #21794: Remove rotation; rely on state activated It seems like we no longer need to use rotation for the chevron, since we are now using two different icons within the `ic_chevon` that change depending on the `state_activated`. --- .../org/mozilla/fenix/tabstray/browser/InactiveTabViewHolder.kt | 2 -- 1 file changed, 2 deletions(-) diff --git a/app/src/main/java/org/mozilla/fenix/tabstray/browser/InactiveTabViewHolder.kt b/app/src/main/java/org/mozilla/fenix/tabstray/browser/InactiveTabViewHolder.kt index 503edc99f..f4346a26c 100644 --- a/app/src/main/java/org/mozilla/fenix/tabstray/browser/InactiveTabViewHolder.kt +++ b/app/src/main/java/org/mozilla/fenix/tabstray/browser/InactiveTabViewHolder.kt @@ -47,7 +47,6 @@ sealed class InactiveTabViewHolder(itemView: View) : RecyclerView.ViewHolder(ite inactiveTabsInteractor.onHeaderClicked(newState) it.isActivated = newState - binding.chevron.rotation = ROTATION_DEGREE correctHeaderBorder(isActivated) } @@ -69,7 +68,6 @@ sealed class InactiveTabViewHolder(itemView: View) : RecyclerView.ViewHolder(ite companion object { const val LAYOUT_ID = R.layout.inactive_header_item - private const val ROTATION_DEGREE = 180F } }