From 5a18c22cbd3e73f2e1be057df58b45cff6d839ed Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Fri, 27 Dec 2019 20:34:07 +0100
Subject: [PATCH] Fixed theme combinations

---
 .../email/FragmentOptionsDisplay.java         | 75 ++++++++++++++-----
 1 file changed, 57 insertions(+), 18 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/FragmentOptionsDisplay.java b/app/src/main/java/eu/faircode/email/FragmentOptionsDisplay.java
index ad9a95d0f8..312984b19f 100644
--- a/app/src/main/java/eu/faircode/email/FragmentOptionsDisplay.java
+++ b/app/src/main/java/eu/faircode/email/FragmentOptionsDisplay.java
@@ -658,39 +658,78 @@ public class FragmentOptionsDisplay extends FragmentBase implements SharedPrefer
     }
 
     public static class FragmentDialogTheme extends FragmentDialogBase {
+        private RadioGroup rgTheme;
+        private SwitchCompat swReverse;
+        private SwitchCompat swDark;
+        private SwitchCompat swSystem;
+
+        private void eval() {
+            int checkedId = rgTheme.getCheckedRadioButtonId();
+
+            swReverse.setEnabled(checkedId == R.id.rbThemeBlueOrange ||
+                    checkedId == R.id.rbThemeYellowPurple ||
+                    checkedId == R.id.rbThemeRedGreen);
+            swDark.setEnabled(checkedId == R.id.rbThemeBlueOrange ||
+                    checkedId == R.id.rbThemeYellowPurple ||
+                    checkedId == R.id.rbThemeRedGreen ||
+                    checkedId == R.id.rbThemeGrey);
+            swSystem.setEnabled(checkedId == R.id.rbThemeBlueOrange ||
+                    checkedId == R.id.rbThemeGrey);
+
+            boolean reverse = (swReverse.isEnabled() && swReverse.isChecked());
+            boolean dark = (swDark.isEnabled() && swDark.isChecked());
+            boolean system = (swSystem.isEnabled() && swSystem.isChecked());
+
+            swReverse.setEnabled(swReverse.isEnabled() && !system);
+            swSystem.setEnabled(swSystem.isEnabled() && !reverse && !dark);
+        }
+
         @NonNull
         @Override
         public Dialog onCreateDialog(@Nullable Bundle savedInstanceState) {
             View dview = LayoutInflater.from(getContext()).inflate(R.layout.dialog_theme, null);
-            final RadioGroup rgTheme = dview.findViewById(R.id.rgTheme);
-            final SwitchCompat swReverse = dview.findViewById(R.id.swReverse);
-            final SwitchCompat swDark = dview.findViewById(R.id.swDark);
-            final SwitchCompat swSystem = dview.findViewById(R.id.swSystem);
+            rgTheme = dview.findViewById(R.id.rgTheme);
+            swReverse = dview.findViewById(R.id.swReverse);
+            swDark = dview.findViewById(R.id.swDark);
+            swSystem = dview.findViewById(R.id.swSystem);
 
             final SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(getContext());
             String theme = prefs.getString("theme", "light");
 
-            swReverse.setChecked(theme.startsWith("orange_blue") ||
-                    theme.startsWith("purple_yellow") ||
-                    theme.startsWith("green_red"));
-            swDark.setChecked(theme.endsWith("dark"));
-            swSystem.setChecked(theme.endsWith("system"));
+            swReverse.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
+                @Override
+                public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
+                    eval();
+                }
+            });
+
+            swDark.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
+                @Override
+                public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
+                    eval();
+                }
+            });
+
+            swSystem.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
+                @Override
+                public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
+                    eval();
+                }
+            });
 
             rgTheme.setOnCheckedChangeListener(new RadioGroup.OnCheckedChangeListener() {
                 @Override
                 public void onCheckedChanged(RadioGroup group, int checkedId) {
-                    swReverse.setEnabled(checkedId == R.id.rbThemeBlueOrange ||
-                            checkedId == R.id.rbThemeYellowPurple ||
-                            checkedId == R.id.rbThemeRedGreen);
-                    swDark.setEnabled(checkedId == R.id.rbThemeBlueOrange ||
-                            checkedId == R.id.rbThemeYellowPurple ||
-                            checkedId == R.id.rbThemeRedGreen ||
-                            checkedId == R.id.rbThemeGrey);
-                    swSystem.setEnabled(checkedId == R.id.rbThemeBlueOrange ||
-                            checkedId == R.id.rbThemeGrey);
+                    eval();
                 }
             });
 
+            swReverse.setChecked(theme.startsWith("orange_blue") ||
+                    theme.startsWith("purple_yellow") ||
+                    theme.startsWith("green_red"));
+            swDark.setChecked(theme.endsWith("dark"));
+            swSystem.setChecked(theme.endsWith("system"));
+
             switch (theme) {
                 case "light":
                 case "dark":