From 4612ceb3c8a2264741d929bb6d60d125f69a81bc Mon Sep 17 00:00:00 2001
From: cketti <cketti@gmail.com>
Date: Tue, 3 Apr 2012 03:41:44 +0200
Subject: [PATCH] Enabled dark theme for MessageCompose and MessageView

---
 .../divider_horizontal_email.9.png            | Bin 434 -> 570 bytes
 res/drawable/divider_horizontal_email.9.png   | Bin 436 -> 570 bytes
 res/layout/message_compose.xml                |  17 ++--------------
 src/com/fsck/k9/activity/K9Activity.java      |   8 +-------
 src/com/fsck/k9/activity/MessageCompose.java  |  15 ++++++++------
 src/com/fsck/k9/activity/MessageView.java     |   2 +-
 src/com/fsck/k9/view/MessageWebView.java      |  19 ++++++++++++++++++
 src/com/fsck/k9/view/SingleMessageView.java   |   2 +-
 8 files changed, 33 insertions(+), 30 deletions(-)

diff --git a/res/drawable-hdpi/divider_horizontal_email.9.png b/res/drawable-hdpi/divider_horizontal_email.9.png
index 8358af9c3eadc38b3d62ba371658a9c60421094f..46f5f48d5fb22c4fe1c5023164b4715bc5f96562 100644
GIT binary patch
literal 570
zcmV-A0>%A_P)<h;3K|Lk000e1NJLTq008m;0009B1^@s6sismG00001b5ch_0Itp)
z=>Px#1ZP1_K>z@;j|==^1poj532;bRa{vGi!vFvd!vV){sAK>D02*{fSaefwW^{L9
za%BKeVQFr3E>1;MAa*k@H7+qQF!XYv00050Nkl<ZNK5V14@<&e5C-r*U_?YjM1({{
zghWV4gai>15)v`z)H$clIp>@?hlE5#M4!v<IqtyEU=w~kbN+kZd+*%Z${w3uffa2K
zvd4y3V6|8*w$n7-f%$yC3rUjf!E83$hv{^B0C5~2!elZzf+&iPVLTq6h#X}u48v0x
zjYek>1i?8Bhr<i-eg9JADf5HD;0ip?yA}n?Lcia?5k<;kuh+YUZnt|UN|Ytnb?<?$
zAAqhOMVYeP>2#h%g|gx}&a<deR&CpUfp)w7Dhx`4?wP`(v|6oJQ`nT?K%>!cq1kNq
zgh%PoJzoTr5Q>O0s@Ll=SeErB5=uzHG|dIjd)~#8a!L0-pjNAW0$qQJZ%X(9!!Uk9
zpUZ1_o!`IqdHz0px8UE?-`DlX+OR%eudW|^5xPI!FYh0<5L_Rvm)1{eiF$H<QES2V
z$5{wIA3ZOeo#6At*$O^yoW0<FSnCh%7w#YJC+#ovn&AFJFADBY?N{`!;C|Nr)_&Li
z*LlEf2%Z<6ADt(iFP%4?Kb=RNPn}oH?uP$AW!uXt@P8`s2X)Uwnq%f{PXGV_07*qo
IM6N<$f_~)vy#N3J

literal 434
zcmV;j0ZsmiP)<h;3K|Lk000e1NJLTq009I50009B1^@s6b$W3I0000PbVXQnQ*UN;
zcVTj606}DLVr3vnZDD6+Qe|Oed2z{QJOBUzR7pfZRCwC#RNIz=APgKJ7XSYz_o?qv
zfoz6ix31d#fMw?7s9Z832?7HHfOZ@B&rP5T;q(4=$HUlAdus%<V&=M)qXTD5MmmZm
zVvz+>1-|+=en9Si@xfnBEry$sKC02R0N(cLabMMnR1$9CX48sXGNN%cW}{Pgskzn_
zr4~f7^~Kg}-fPxwCs^`=-~z-;dq3bJ-l7GM@@SgWW(*u>%*D8>eHV*CuJw7&loO^f
zwY~JmG5R20HQmk~dJR7R81|4~`|+#|!TL~l%Q<t$^XziB!FMx#4T{N4<}N(@$2fWW
z_Mx4J{+LreChn(y-Z0CITyx7Q-%qjWzvWJxwA@jX+KfTZ{)?umj+e#O8yT|%dF4EY
z2=Dx#@p9f9S)W!LizoVcsrjm%9H;4{`ikr`<~$<>tA%^^&!s;-x!ZnMQ|EY@2L=WP
ch;IP~0CbSBzQWjTL;wH)07*qoM6N<$f*^g%5&!@I

diff --git a/res/drawable/divider_horizontal_email.9.png b/res/drawable/divider_horizontal_email.9.png
index 1f56364f274656e82667a81ae777788228a64c9d..46f5f48d5fb22c4fe1c5023164b4715bc5f96562 100644
GIT binary patch
literal 570
zcmV-A0>%A_P)<h;3K|Lk000e1NJLTq008m;0009B1^@s6sismG00001b5ch_0Itp)
z=>Px#1ZP1_K>z@;j|==^1poj532;bRa{vGi!vFvd!vV){sAK>D02*{fSaefwW^{L9
za%BKeVQFr3E>1;MAa*k@H7+qQF!XYv00050Nkl<ZNK5V14@<&e5C-r*U_?YjM1({{
zghWV4gai>15)v`z)H$clIp>@?hlE5#M4!v<IqtyEU=w~kbN+kZd+*%Z${w3uffa2K
zvd4y3V6|8*w$n7-f%$yC3rUjf!E83$hv{^B0C5~2!elZzf+&iPVLTq6h#X}u48v0x
zjYek>1i?8Bhr<i-eg9JADf5HD;0ip?yA}n?Lcia?5k<;kuh+YUZnt|UN|Ytnb?<?$
zAAqhOMVYeP>2#h%g|gx}&a<deR&CpUfp)w7Dhx`4?wP`(v|6oJQ`nT?K%>!cq1kNq
zgh%PoJzoTr5Q>O0s@Ll=SeErB5=uzHG|dIjd)~#8a!L0-pjNAW0$qQJZ%X(9!!Uk9
zpUZ1_o!`IqdHz0px8UE?-`DlX+OR%eudW|^5xPI!FYh0<5L_Rvm)1{eiF$H<QES2V
z$5{wIA3ZOeo#6At*$O^yoW0<FSnCh%7w#YJC+#ovn&AFJFADBY?N{`!;C|Nr)_&Li
z*LlEf2%Z<6ADt(iFP%4?Kb=RNPn}oH?uP$AW!uXt@P8`s2X)Uwnq%f{PXGV_07*qo
IM6N<$f_~)vy#N3J

literal 436
zcmV;l0ZaagP)<h;3K|Lk000e1NJLTq009I50009B1^@s6b$W3I0000PbVXQnQ*UN;
zcVTj606}DLVr3vnZDD6+Qe|Oed2z{QJOBUzR!KxbRCwC#RNI!rAP58{HUIyU`_%W?
zh%94mn>NYz1J;@2iSfc9piBb;MApANLh+A_z>Ean>(7;L&W75{BYM+bBHa5QU}tv1
zP|`$;EF@JZ)X>IvDBUjoVXx-q!^H_3)ydmB)lK<s_EoEtHPa<teA=jJq2yeh#mUB}
znrmH2t5OnQufE>OyJqd}083dYawG}T`a7KXtEP0*ThoL#e84z-R_AK|ukM4K>*Jic
zWSZjC_QD><U;}vO^l|RkbMXCPctJk)(>*r;@&nyR%n>`BhmYQs(3|U9G@5vFbMan3
z&MEkRi0wSEhnU(vaIgB=V3wImEiI<#AL60EiJh>t*g+H8@WBp308L|D7GLj#&l2Ss
z^Ar;W4qyGnd~hN^EjR8@3h7kynLF7}!$u92N+>j!nR4`8@V$Oc?b*TI4sSVS#$_HD
e7#IM41sDKrcdx;_N6PE~0000<MNUMnLSTYH<juGM

diff --git a/res/layout/message_compose.xml b/res/layout/message_compose.xml
index f22928a56..826040c99 100644
--- a/res/layout/message_compose.xml
+++ b/res/layout/message_compose.xml
@@ -2,8 +2,7 @@
 <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
     android:layout_width="fill_parent"
     android:layout_height="fill_parent"
-    android:orientation="vertical"
-    android:background="#ffffff" >
+    android:orientation="vertical">
 
     <ScrollView
         android:layout_width="fill_parent"
@@ -23,7 +22,7 @@
                 android:layout_marginBottom="4dp"
                 android:paddingTop="6dp"
                 android:orientation="vertical"
-                android:background="#ededed" >
+                android:background="#45bcbcbc">
 
                 <Button
                     android:id="@+id/identity"
@@ -47,7 +46,6 @@
                         android:inputType="textEmailAddress|textMultiLine"
                         android:imeOptions="actionNext"
                         android:hint="@string/message_compose_to_hint"
-                        android:textColor="@android:color/primary_text_light"
                         android:textAppearance="?android:attr/textAppearanceMedium"
                         android:layout_width="0dp"
                         android:layout_marginRight="6dip"
@@ -81,7 +79,6 @@
                         android:inputType="textEmailAddress|textMultiLine"
                         android:imeOptions="actionNext"
                         android:hint="@string/message_compose_cc_hint"
-                        android:textColor="@android:color/primary_text_light"
                         android:textAppearance="?android:attr/textAppearanceMedium"/>
 
                     <ImageButton
@@ -112,7 +109,6 @@
                         android:inputType="textEmailAddress|textMultiLine"
                         android:imeOptions="actionNext"
                         android:hint="@string/message_compose_bcc_hint"
-                        android:textColor="@android:color/primary_text_light"
                         android:textAppearance="?android:attr/textAppearanceMedium"/>
 
                     <ImageButton
@@ -144,7 +140,6 @@
                         <CheckBox
                             android:text="@string/btn_crypto_sign"
                             android:id="@+id/cb_crypto_signature"
-                            android:textColor="@android:color/primary_text_light"
                             android:layout_gravity="center_vertical"
                             android:layout_width="wrap_content"
                             android:layout_height="wrap_content"/>
@@ -160,7 +155,6 @@
                                 android:id="@+id/userId"
                                 android:ellipsize="end"
                                 android:textAppearance="?android:attr/textAppearanceSmall"
-                                android:textColor="@android:color/primary_text_light"
                                 android:layout_width="wrap_content"
                                 android:layout_height="wrap_content"/>
 
@@ -168,7 +162,6 @@
                                 android:id="@+id/userIdRest"
                                 android:textSize="10sp"
                                 android:ellipsize="end"
-                                android:textColor="@android:color/primary_text_light"
                                 android:layout_width="wrap_content"
                                 android:layout_height="wrap_content"/>
 
@@ -179,7 +172,6 @@
                     <CheckBox
                         android:text="@string/btn_encrypt"
                         android:id="@+id/cb_encrypt"
-                        android:textColor="@android:color/primary_text_light"
                         android:layout_gravity="center_vertical"
                         android:layout_width="wrap_content"
                         android:layout_height="wrap_content"
@@ -197,7 +189,6 @@
                     android:inputType="textEmailSubject|textAutoCorrect|textCapSentences|textImeMultiLine"
                     android:imeOptions="actionNext"
                     android:singleLine="true"
-                    android:textColor="@android:color/primary_text_light"
                     android:textAppearance="?android:attr/textAppearanceMedium" />
 
                 <!--
@@ -229,7 +220,6 @@
                 android:inputType="textMultiLine|textAutoCorrect|textCapSentences"
                 android:imeOptions="actionDone|flagNoEnterAction"
                 android:minLines="3"
-                android:textColor="@android:color/primary_text_light"
                 android:textAppearance="?android:attr/textAppearanceMedium" />
 
             <EditText
@@ -243,7 +233,6 @@
                 android:capitalize="sentences"
                 android:hint="@string/message_compose_signature_hint"
                 android:inputType="textMultiLine|textAutoCorrect|textCapSentences"
-                android:textColor="@android:color/primary_text_light"
                 android:textAppearance="?android:attr/textAppearanceMedium" />
 
             <Button
@@ -270,7 +259,6 @@
                     android:autoText="true"
                     android:capitalize="sentences"
                     android:inputType="textMultiLine|textAutoCorrect|textCapSentences"
-                    android:textColor="@android:color/primary_text_light"
                     android:textAppearance="?android:attr/textAppearanceMedium" />
 
                 <com.fsck.k9.view.MessageWebView
@@ -315,7 +303,6 @@
                 android:capitalize="sentences"
                 android:hint="@string/message_compose_signature_hint"
                 android:inputType="textMultiLine|textAutoCorrect|textCapSentences"
-                android:textColor="@android:color/primary_text_light"
                 android:textAppearance="?android:attr/textAppearanceMedium" />
 
         </LinearLayout>
diff --git a/src/com/fsck/k9/activity/K9Activity.java b/src/com/fsck/k9/activity/K9Activity.java
index 1c1a26eab..c346ab71c 100644
--- a/src/com/fsck/k9/activity/K9Activity.java
+++ b/src/com/fsck/k9/activity/K9Activity.java
@@ -23,14 +23,8 @@ public class K9Activity extends Activity {
 
     @Override
     public void onCreate(Bundle icicle) {
-        onCreate(icicle, true);
-    }
-
-    public void onCreate(Bundle icicle, boolean useTheme) {
         setLanguage(this, K9.getK9Language());
-        if (useTheme) {
-            setTheme(K9.getK9ThemeResourceId());
-        }
+        setTheme(K9.getK9ThemeResourceId());
         super.onCreate(icicle);
         setupFormats();
 
diff --git a/src/com/fsck/k9/activity/MessageCompose.java b/src/com/fsck/k9/activity/MessageCompose.java
index 19f94e0c3..e02186771 100644
--- a/src/com/fsck/k9/activity/MessageCompose.java
+++ b/src/com/fsck/k9/activity/MessageCompose.java
@@ -19,12 +19,14 @@ import android.app.AlertDialog.Builder;
 import android.app.Dialog;
 import android.content.ContentResolver;
 import android.content.Context;
+import android.content.ContextWrapper;
 import android.content.DialogInterface;
 import android.content.Intent;
 import android.content.pm.ActivityInfo;
 import android.database.Cursor;
 import android.net.Uri;
 import android.os.AsyncTask;
+import android.os.Build;
 import android.os.Bundle;
 import android.os.Handler;
 import android.os.Parcelable;
@@ -377,7 +379,6 @@ public class MessageCompose extends K9Activity implements OnClickListener, OnFoc
     @Override
     public void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
-        setTheme(K9.getK9ThemeResourceId(K9.THEME_LIGHT));
         requestWindowFeature(Window.FEATURE_INDETERMINATE_PROGRESS);
         setContentView(R.layout.message_compose);
 
@@ -952,7 +953,7 @@ public class MessageCompose extends K9Activity implements OnClickListener, OnFoc
         if (mQuotedTextMode != QuotedTextMode.NONE && mMessageFormat == MessageFormat.HTML) {
             mQuotedHtmlContent = (InsertableHtmlContent) savedInstanceState.getSerializable(STATE_KEY_HTML_QUOTE);
             if (mQuotedHtmlContent != null && mQuotedHtmlContent.getQuotedContent() != null) {
-                mQuotedHTML.loadDataWithBaseURL("http://", mQuotedHtmlContent.getQuotedContent(), "text/html", "utf-8", null);
+                mQuotedHTML.setText(mQuotedHtmlContent.getQuotedContent(), "text/html");
             }
         }
         mDraftId = savedInstanceState.getLong(STATE_KEY_DRAFT_ID);
@@ -2119,9 +2120,11 @@ public class MessageCompose extends K9Activity implements OnClickListener, OnFoc
             })
             .create();
         case DIALOG_CHOOSE_IDENTITY:
-            Builder builder = new AlertDialog.Builder(this);
+            Context context = new ContextWrapper(this);
+            context.setTheme(K9.getK9ThemeResourceId(K9.THEME_LIGHT));
+            Builder builder = new AlertDialog.Builder(context);
             builder.setTitle(R.string.send_as);
-            final IdentityAdapter adapter = new IdentityAdapter(this, getLayoutInflater());
+            final IdentityAdapter adapter = new IdentityAdapter(context, getLayoutInflater());
             builder.setAdapter(adapter, new DialogInterface.OnClickListener() {
                 @Override
                 public void onClick(DialogInterface dialog, int which) {
@@ -2480,7 +2483,7 @@ public class MessageCompose extends K9Activity implements OnClickListener, OnFoc
                             } else {
                                 mQuotedHtmlContent.setFooterInsertionPoint(bodyOffset);
                             }
-                            mQuotedHTML.loadDataWithBaseURL("http://", mQuotedHtmlContent.getQuotedContent(), "text/html", "utf-8", null);
+                            mQuotedHTML.setText(mQuotedHtmlContent.getQuotedContent(), "text/html");
                         }
                     }
                     if (bodyPlainOffset != null && bodyPlainLength != null) {
@@ -2663,7 +2666,7 @@ public class MessageCompose extends K9Activity implements OnClickListener, OnFoc
             // Add the HTML reply header to the top of the content.
             mQuotedHtmlContent = quoteOriginalHtmlMessage(mSourceMessage, content, mQuoteStyle);
             // Load the message with the reply header.
-            mQuotedHTML.loadDataWithBaseURL("http://", mQuotedHtmlContent.getQuotedContent(), "text/html", "utf-8", null);
+            mQuotedHTML.setText(mQuotedHtmlContent.getQuotedContent(), "text/html");
             mQuotedText.setText(quoteOriginalTextMessage(mSourceMessage,
                     getBodyTextFromMessage(mSourceMessage, MessageFormat.TEXT), mQuoteStyle));
 
diff --git a/src/com/fsck/k9/activity/MessageView.java b/src/com/fsck/k9/activity/MessageView.java
index 87d968681..91ea29690 100644
--- a/src/com/fsck/k9/activity/MessageView.java
+++ b/src/com/fsck/k9/activity/MessageView.java
@@ -287,7 +287,7 @@ public class MessageView extends K9Activity implements OnClickListener {
 
     @Override
     public void onCreate(Bundle icicle) {
-        super.onCreate(icicle, false);
+        super.onCreate(icicle);
         requestWindowFeature(Window.FEATURE_INDETERMINATE_PROGRESS);
         requestWindowFeature(Window.FEATURE_NO_TITLE);
         setContentView(R.layout.message_view);
diff --git a/src/com/fsck/k9/view/MessageWebView.java b/src/com/fsck/k9/view/MessageWebView.java
index d992b50be..6ec7f50bb 100644
--- a/src/com/fsck/k9/view/MessageWebView.java
+++ b/src/com/fsck/k9/view/MessageWebView.java
@@ -72,6 +72,12 @@ public class MessageWebView extends WebView {
         this.setScrollBarStyle(SCROLLBARS_INSIDE_OVERLAY);
         this.setLongClickable(true);
 
+        if (K9.getK9Theme() == K9.THEME_DARK) {
+            // Black theme should get a black webview background
+            // we'll set the background of the messages on load
+            this.setBackgroundColor(0xff000000);
+        }
+
         final WebSettings webSettings = this.getSettings();
 
         webSettings.setCacheMode(WebSettings.LOAD_NO_CACHE);
@@ -104,6 +110,19 @@ public class MessageWebView extends WebView {
 
     }
 
+    public void setText(String text, String contentType) {
+        String content = text;
+        if (K9.getK9Theme() == K9.THEME_DARK)  {
+            // It's a little wrong to just throw in the <style> before the opening <html>
+            // but it's less wrong than trying to edit the html stream
+            content = "<style>* { background: black ! important; color: white !important }" +
+                   ":link, :link * { color: #CCFF33 !important }" +
+                   ":visited, :visited * { color: #551A8B !important }</style> "
+                   + content;
+        }
+        loadDataWithBaseURL("http://", content, contentType, "utf-8", null);
+    }
+
     /*
      * Emulate the shift key being pressed to trigger the text selection mode
      * of a WebView.
diff --git a/src/com/fsck/k9/view/SingleMessageView.java b/src/com/fsck/k9/view/SingleMessageView.java
index 3dc1c92a1..54b65b07b 100644
--- a/src/com/fsck/k9/view/SingleMessageView.java
+++ b/src/com/fsck/k9/view/SingleMessageView.java
@@ -600,7 +600,7 @@ public class SingleMessageView extends LinearLayout implements OnClickListener,
         if (mScreenReaderEnabled) {
             mAccessibleMessageContentView.loadDataWithBaseURL("http://", emailText, contentType, "utf-8", null);
         } else {
-            mMessageContentView.loadDataWithBaseURL("http://", emailText, contentType, "utf-8", null);
+            mMessageContentView.setText(emailText, contentType);
             mMessageContentView.scrollTo(0, 0);
         }