From 59bd4ec833232e73226755dc1e1076d0885226bb Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Thu, 2 Jan 2020 21:27:00 +0100
Subject: [PATCH] Revert "Always set card background color"

This reverts commit 9ca9a864168b5da918e023315f50ab9203ed6313.
---
 .../java/eu/faircode/email/ViewCardOptional.java  | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/ViewCardOptional.java b/app/src/main/java/eu/faircode/email/ViewCardOptional.java
index 83bf7d435e..3f4661fcd2 100644
--- a/app/src/main/java/eu/faircode/email/ViewCardOptional.java
+++ b/app/src/main/java/eu/faircode/email/ViewCardOptional.java
@@ -37,6 +37,7 @@ public class ViewCardOptional extends CardView {
     private boolean threading;
     private int margin;
     private int ident;
+    private Integer color = null;
 
     public ViewCardOptional(@NonNull Context context) {
         super(context);
@@ -82,12 +83,16 @@ public class ViewCardOptional extends CardView {
 
     @Override
     public void setCardBackgroundColor(int color) {
-        SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(getContext());
-        boolean cards = prefs.getBoolean("cards", true);
-        if (cards && color == Color.TRANSPARENT)
-            color = Helper.resolveColor(getContext(), R.attr.colorCardBackground);
+        if (this.color == null || this.color != color) {
+            this.color = color;
 
-        super.setCardBackgroundColor(color);
+            SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(getContext());
+            boolean cards = prefs.getBoolean("cards", true);
+            if (cards && color == Color.TRANSPARENT)
+                color = Helper.resolveColor(getContext(), R.attr.colorCardBackground);
+
+            super.setCardBackgroundColor(color);
+        }
     }
 
     public void setOutgoing(boolean outgoing) {