From 4ac61a7d344031208aa08e2d2f054b3bd54cbc54 Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Sat, 20 Apr 2019 16:00:46 +0200
Subject: [PATCH] Added option to enable inverting original message

---
 FAQ.md                                             |  4 +++-
 .../java/eu/faircode/email/AdapterMessage.java     | 13 +++++--------
 .../java/eu/faircode/email/FragmentOptions.java    | 14 ++++++++++++--
 app/src/main/java/eu/faircode/email/Helper.java    |  7 +++++++
 app/src/main/res/layout/fragment_options.xml       | 14 +++++++++++++-
 app/src/main/res/values/strings.xml                |  1 +
 6 files changed, 41 insertions(+), 12 deletions(-)

diff --git a/FAQ.md b/FAQ.md
index 5f6a8581f3..128bdf17ca 100644
--- a/FAQ.md
+++ b/FAQ.md
@@ -1391,7 +1391,9 @@ FairEmail already tries to workaround these bugs, but if this fail you'll need t
 ~~The original message is shown as the sender has sent it, including all colors.~~
 ~~Changing the background color would not only make the original view not original anymore, it can also result in unreadable messages.~~
 
-Recent versions of FairEmail will invert all colors of the original message when using a dark or black theme.
+Recent versions of FairEmail can invert all colors of the original message when using a dark or black theme.
+You can turn this on the advanced settings.
+Be aware that this can cause [crashes](https://bugs.chromium.org/p/chromium/issues/detail?id=578150) on some devices.
 
 <br />
 
diff --git a/app/src/main/java/eu/faircode/email/AdapterMessage.java b/app/src/main/java/eu/faircode/email/AdapterMessage.java
index ec16d4014b..eb564bc956 100644
--- a/app/src/main/java/eu/faircode/email/AdapterMessage.java
+++ b/app/src/main/java/eu/faircode/email/AdapterMessage.java
@@ -1400,16 +1400,13 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
                     }
                 };
 
-                if (dark) {
-                    float[] NEGATIVE = new float[]{
-                            -1, 0, 0, 0, 255, // red
-                            0, -1, 0, 0, 255, // green
-                            0, 0, -1, 0, 255, // blue
-                            0, 0, 0, 1, 0 // alpha
-                    };
+                SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(context);
+                boolean invert = prefs.getBoolean("invert", false);
 
+                if (dark && invert) {
+                    // https://bugs.chromium.org/p/chromium/issues/detail?id=578150
                     Paint paint = new Paint();
-                    paint.setColorFilter(new ColorMatrixColorFilter(NEGATIVE));
+                    paint.setColorFilter(new ColorMatrixColorFilter(Helper.MATRIX_NEGATIVE));
                     webView.setLayerType(View.LAYER_TYPE_HARDWARE, paint);
                 }
 
diff --git a/app/src/main/java/eu/faircode/email/FragmentOptions.java b/app/src/main/java/eu/faircode/email/FragmentOptions.java
index b07f356b0f..57a98a99b7 100644
--- a/app/src/main/java/eu/faircode/email/FragmentOptions.java
+++ b/app/src/main/java/eu/faircode/email/FragmentOptions.java
@@ -86,6 +86,7 @@ public class FragmentOptions extends FragmentBase implements SharedPreferences.O
     private SwitchCompat swMonospaced;
     private SwitchCompat swHtml;
     private SwitchCompat swImages;
+    private SwitchCompat swInvert;
     private SwitchCompat swActionbar;
 
     private SwitchCompat swPull;
@@ -120,7 +121,7 @@ public class FragmentOptions extends FragmentBase implements SharedPreferences.O
 
     static String[] OPTIONS_RESTART = new String[]{
             "startup", "date", "threading", "avatars", "identicons", "name_email", "subject_italic", "flags", "preview",
-            "addresses", "monospaced", "autohtml", "autoimages", "actionbar",
+            "addresses", "monospaced", "autohtml", "autoimages", "invert", "actionbar",
             "pull", "swipenav", "autoexpand", "autoclose", "autonext",
             "authentication", "debug"
     };
@@ -129,7 +130,7 @@ public class FragmentOptions extends FragmentBase implements SharedPreferences.O
             "enabled", "schedule_start", "schedule_end",
             "metered", "download", "roaming",
             "startup", "date", "threading", "avatars", "identicons", "name_email", "subject_italic", "flags", "preview",
-            "addresses", "monospaced", "autohtml", "autoimages", "actionbar",
+            "addresses", "monospaced", "autohtml", "autoimages", "invert", "actionbar",
             "pull", "swipenav", "autoexpand", "autoclose", "autonext", "collapse", "autoread", "automove",
             "autoresize", "resize", "prefix_once", "autosend",
             "notify_preview", "search_local", "light", "sound",
@@ -171,6 +172,7 @@ public class FragmentOptions extends FragmentBase implements SharedPreferences.O
         swMonospaced = view.findViewById(R.id.swMonospaced);
         swHtml = view.findViewById(R.id.swHtml);
         swImages = view.findViewById(R.id.swImages);
+        swInvert = view.findViewById(R.id.swInvert);
         swActionbar = view.findViewById(R.id.swActionbar);
 
         swPull = view.findViewById(R.id.swPull);
@@ -390,6 +392,13 @@ public class FragmentOptions extends FragmentBase implements SharedPreferences.O
             }
         });
 
+        swInvert.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
+            @Override
+            public void onCheckedChanged(CompoundButton compoundButton, boolean checked) {
+                prefs.edit().putBoolean("invert", checked).apply();
+            }
+        });
+
         swActionbar.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
             @Override
             public void onCheckedChanged(CompoundButton compoundButton, boolean checked) {
@@ -686,6 +695,7 @@ public class FragmentOptions extends FragmentBase implements SharedPreferences.O
         swMonospaced.setChecked(prefs.getBoolean("monospaced", false));
         swHtml.setChecked(prefs.getBoolean("autohtml", false));
         swImages.setChecked(prefs.getBoolean("autoimages", false));
+        swInvert.setChecked(prefs.getBoolean("invert", false));
         swActionbar.setChecked(prefs.getBoolean("actionbar", true));
 
         swPull.setChecked(prefs.getBoolean("pull", true));
diff --git a/app/src/main/java/eu/faircode/email/Helper.java b/app/src/main/java/eu/faircode/email/Helper.java
index d6adfa8ecd..e96b009837 100644
--- a/app/src/main/java/eu/faircode/email/Helper.java
+++ b/app/src/main/java/eu/faircode/email/Helper.java
@@ -132,6 +132,13 @@ public class Helper {
 
     static final float LOW_LIGHT = 0.6f;
 
+    static final float[] MATRIX_NEGATIVE = new float[]{
+            -1, 0, 0, 0, 255, // red
+            0, -1, 0, 0, 255, // green
+            0, 0, -1, 0, 255, // blue
+            0, 0, 0, 1, 0 // alpha
+    };
+
     static final String FAQ_URI = "https://github.com/M66B/open-source-email/blob/master/FAQ.md";
 
     static ThreadFactory backgroundThreadFactory = new ThreadFactory() {
diff --git a/app/src/main/res/layout/fragment_options.xml b/app/src/main/res/layout/fragment_options.xml
index afb58f7da5..34fab46a96 100644
--- a/app/src/main/res/layout/fragment_options.xml
+++ b/app/src/main/res/layout/fragment_options.xml
@@ -503,6 +503,18 @@
             app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintTop_toBottomOf="@id/swImages" />
 
+        <androidx.appcompat.widget.SwitchCompat
+            android:id="@+id/swInvert"
+            android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:layout_marginStart="12dp"
+            android:layout_marginTop="12dp"
+            android:layout_marginEnd="12dp"
+            android:text="@string/title_advanced_invert"
+            app:layout_constraintStart_toStartOf="parent"
+            app:layout_constraintTop_toBottomOf="@id/tvImagesHint"
+            app:switchPadding="12dp" />
+
         <androidx.appcompat.widget.SwitchCompat
             android:id="@+id/swActionbar"
             android:layout_width="match_parent"
@@ -512,7 +524,7 @@
             android:layout_marginEnd="12dp"
             android:text="@string/title_advanced_actionbar"
             app:layout_constraintStart_toStartOf="parent"
-            app:layout_constraintTop_toBottomOf="@id/tvImagesHint"
+            app:layout_constraintTop_toBottomOf="@id/swInvert"
             app:switchPadding="12dp" />
 
         <TextView
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 77413cdb81..7077fc4965 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -177,6 +177,7 @@
     <string name="title_advanced_monospaced">Use monospaced font for message text</string>
     <string name="title_advanced_html">Automatically show original message for known contacts</string>
     <string name="title_advanced_images">Automatically show images for known contacts</string>
+    <string name="title_advanced_invert">Invert colors of original message when using a dark theme</string>
     <string name="title_advanced_actionbar">Conversation action bar</string>
 
     <string name="title_advanced_pull_refresh">Pull down to refresh</string>