Fix empty subject header display (#1485383) + improve safari CSS hacks

program/steps/mail/func.inc

@@ -774,10 +774,6 @@ function rcmail_message_headers($attrib, $headers=NULL)
   if (!$headers)
     $headers = is_object($MESSAGE->headers) ? get_object_vars($MESSAGE->headers) : $MESSAGE->headers;
     
-  // add empty subject if none exsists
-  if (empty($headers['subject']))
-    $headers['subject'] = rcube_label('nosubject');
-
   $header_count = 0;
   
   // allow the following attributes to be added to the <table> tag
@@ -808,6 +804,8 @@ function rcmail_message_headers($attrib, $headers=NULL)
       }
     else if (in_array($hkey, array('from', 'to', 'cc', 'bcc')))
       $header_value = Q(rcmail_address_string($headers[$hkey], null, true, $attrib['addicon']), 'show');
+    else if ($hkey == 'subject' && empty($headers[$hkey]))
+      $header_value = Q(rcube_label('nosubject'));
     else
       $header_value = Q(trim($IMAP->decode_header($headers[$hkey])));
 

skins/default/mail.css

@@ -440,11 +440,6 @@ body.messagelist
   width: expression(parseInt(document.getElementById('mailcontframe').clientWidth)+'px');
 }
 
-/* safari hack \*/
-html>body*#messagelist[id$="messagelist"]:not([class="none"]) { width: 99%; }
-html>body*input[type$="file"]:not([class="none"]) { background-color: transparent; border:0; }
-/**/
-
 #messagelist thead tr td
 {
   height: 20px;
@@ -604,6 +599,12 @@ html>body*input[type$="file"]:not([class="none"]) { background-color: transparen
   color: #CCCCCC;
 }
 
+/* safari hacks \*/
+html>body*#messagelist[id$="messagelist"]:not([class="none"]) { width: 99.8%; }
+html>body*#messagelist[id$="messagelist"]:not([class="none"]) tr td.flag,
+html>body*#messagelist[id$="messagelist"]:not([class="none"]) tr td.icon { width: 20px; }
+html>body*input[type$="file"]:not([class="none"]) { background-color: transparent; border: 0; }
+/**/
 
 #quotadisplay
 {