Use <pre> instead of <div class=pre>: Fix line-break issue when copy …

…& paste in Firefox (#1485425)
roundcube · Dec 30, 2008 · 2780647 · 2780647
1 parent 64db7e1
commit 2780647
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/program/steps/mail/func.inc b/program/steps/mail/func.inc
@@ -755,8 +755,8 @@ function rcmail_print_body($part, $p = array())
 
   // insert the links for urls and mailtos
   $body = preg_replace("/##string_replacement\{([0-9]+)\}##/e", "\$replace_strings[\\1]", join("\n", $a_lines));
-  
-  return "<div class=\"pre\">".$body."\n</div>";
+
+  return html::tag('pre', array(), $body);
 }
 
 /**
@@ -924,7 +924,7 @@ function rcmail_message_body($attrib)
       }
     }
   else
-    $out .= html::div('message-part', html::div('pre', Q($MESSAGE->body)));
+    $out .= html::div('message-part', html::tag('pre', array(), Q($MESSAGE->body)));
 
 
   $ctype_primary = strtolower($MESSAGE->structure->ctype_primary);

diff --git a/skins/default/mail.css b/skins/default/mail.css
@@ -820,6 +820,7 @@ div.message-htmlpart a
   color: #0000CC;
 }
 
+div.message-part pre,
 div.message-part div.pre
 {
   margin: 0px;