From b4d396c05d7f49be4947f3efc0400f998e196c5e Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Wed, 30 Sep 2020 19:53:18 +0200
Subject: [PATCH] Simplification

---
 .../java/eu/faircode/email/FragmentOptionsDisplay.java     | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/FragmentOptionsDisplay.java b/app/src/main/java/eu/faircode/email/FragmentOptionsDisplay.java
index 5c91b57a41..115daa0a82 100644
--- a/app/src/main/java/eu/faircode/email/FragmentOptionsDisplay.java
+++ b/app/src/main/java/eu/faircode/email/FragmentOptionsDisplay.java
@@ -987,7 +987,7 @@ public class FragmentOptionsDisplay extends FragmentBase implements SharedPrefer
 
             swBlack.setEnabled(colored && !grey && optionId != R.id.rbThemeLight);
 
-            tvSystem.setEnabled(rgThemeOptions.isEnabled() && optionId == R.id.rbThemeSystem);
+            tvSystem.setEnabled(colored && optionId == R.id.rbThemeSystem);
         }
 
         @NonNull
@@ -1122,9 +1122,10 @@ public class FragmentOptionsDisplay extends FragmentBase implements SharedPrefer
 
                             ContactInfo.clearCache(getContext());
 
+                            int optionId = rgThemeOptions.getCheckedRadioButtonId();
                             boolean reverse = (swReverse.isEnabled() && swReverse.isChecked());
-                            boolean dark = (rgThemeOptions.isEnabled() && rgThemeOptions.getCheckedRadioButtonId() == R.id.rbThemeDark);
-                            boolean system = (rgThemeOptions.isEnabled() && rgThemeOptions.getCheckedRadioButtonId() == R.id.rbThemeSystem);
+                            boolean dark = (rgThemeOptions.isEnabled() && optionId == R.id.rbThemeDark);
+                            boolean system = (rgThemeOptions.isEnabled() && optionId == R.id.rbThemeSystem);
                             boolean black = (swBlack.isEnabled() && swBlack.isChecked());
 
                             switch (rgTheme.getCheckedRadioButtonId()) {