User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:14.0) Gecko/20120316 Firefox/14.0a1 Build ID: 20120316031151 Steps to reproduce: I have this message in my sentmail folder: ------------- Hello, folks - I'm just following up about the tech demo I showed you in late January. Have you come to any conclusions about it? Thanks, -- Michael Hoye Bespoke I/O http://bespokeio.com ------------- and when I click "Forward", I see this: ------------- -------- Original Message -------- Subject: Re: WebFWD Intro Date: Tue, 14 Feb 2012 11:34:55 -0500 From: Mike Hoye <mhoye@bespokeio.com> To: Pascal Finette | Mozilla <pfinette@mozilla.com> CC: Diane Bisgeier <dbisgeier@mozilla.com> Hello, folks - Thanks, I'm just following up about the tech demo I showed you in late January. Have you come to any conclusions about it? -- Michael HoyeBespoke I/Ohttp://bespokeio.com ------------- Because wat. Actual results: http://pastebin.mozilla.org/1522640 Expected results: Not that.

First run at a patch.

Comment on attachment 606708 [details] [diff] [review] Patch v1 It's funny - the bug I was seeing before, where carriage returns were being truncated...I'm not seeing it now. I wonder if the changes made to nsMsgCompose.cpp fixed that. :/ Regardless, this *seems* to fix things. Do you have cycles to test / review this, Jim?

A workaround for now is to forward mail using the HTML editor (press and hold Shift while clicking Forward to do this).

Confirmed with today's trunk nightly. Forwarding as attachment is broken as well, I'll file a separate bug on that.

(confirmed that it's broken badly, I mean, not that the patch works - the message is basically completely rearranged when forwarding inline...)

Comment on attachment 606708 [details] [diff] [review] Patch v1 Review of attachment 606708 [details] [diff] [review]: ----------------------------------------------------------------- This works for me, too.

Committed to comm-central as http://hg.mozilla.org/comm-central/rev/c212bdbccf39

rsx11m: When tonight's Daily gets spun out, mind taking it for a spin to see if we've fully resolved the issue? Thanks! -Mike

Pushed to comm-aurora as http://hg.mozilla.org/releases/comm-aurora/rev/5924288478d6

Looking at Thunderbird/14.0a1 BuildID=20120319030043 on Windows 7, the main issue seems to be fixed with today's nightly. The spacing in the forwarded part matches the original spacing. However, there is an additional blank line now at the very top of the plain-text composition window (i.e., three blank lines before the "---Original Message---" heading instead of two as before). This may be unrelated to the bug here though, given that a simple "Write" opens the composition window with an additional blank line on top in the editor which wasn't there before.