From f7b2bfba092213af2f7101b18b96e957cbe0b217 Mon Sep 17 00:00:00 2001
From: Aleksander Machniak <alec@alec.pl>
Date: Sat, 21 Sep 2013 05:24:16 -0400
Subject: [PATCH] Bring back possibility to unset default font family and font size Fix style attribute quoting when font-family contains double quotes SOme code improvements

---
 program/steps/mail/sendmail.inc |   16 +++++++++++-----
 1 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/program/steps/mail/sendmail.inc b/program/steps/mail/sendmail.inc
index 3f4475e..dee8d21 100644
--- a/program/steps/mail/sendmail.inc
+++ b/program/steps/mail/sendmail.inc
@@ -473,13 +473,19 @@
 $message_body = get_input_value('_message', RCUBE_INPUT_POST, TRUE, $message_charset);
 
 if ($isHtml) {
-  $font_family = rcube_fontdefs($RCMAIL->config->get('default_font', 'Arial'));
-  $font_size = $RCMAIL->config->get('default_font_size');
-  $bstyle = ' style="font:' . $font_size . ' ' . $font_family . ';"';
+  $bstyle = array();
+
+  if ($font_size = $RCMAIL->config->get('default_font_size')) {
+    $bstyle[] = 'font-size: ' . $font_size;
+  }
+  if ($font_family = $RCMAIL->config->get('default_font')) {
+    $bstyle[] = 'font-family: ' . rcmail::font_defs($font_family);
+  }
 
   // append doctype and html/body wrappers
-  $message_body = '<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN">' .
-    "\r\n<html><body$bstyle>\r\n" . $message_body;
+  $message_body = '<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN">'
+    . "\r\n<html><body" . (!empty($bstyle) ? " style='" . implode($bstyle, '; ') . "'" : '') . ">\r\n"
+    . $message_body;
 }
 
 if (!$savedraft) {

--
Gitblit v1.9.1