Closed
Bug 250291
Opened 20 years ago
Closed 20 years ago
html messages: forward as inline not quoting headers or inserting "original message" header
Categories
(MailNews Core :: Composition, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: sspitzer, Assigned: neil)
References
Details
(Keywords: regression, Whiteboard: fixed-aviary1.0)
Attachments
(1 file)
|
(deleted),
patch
|
mscott
:
review+
Bienvenu
:
superreview+
sspitzer
:
approval1.8a2+
|
Details | Diff | Splinter Review |
forward as inline not quoting headers for html messages.
it seems to be working for plain text messages.
note, the mozilla 1.7 and aviary 1.0 branches don't seem to be affected
|
Reporter | |
Comment 1•20 years ago
|
||
for html message, I'm not seeing this text:
-------- Original Message --------
Subject: abc
...
No longer blocks: 249882
Summary: forward as inline not quoting headers for html messages → html messages: forward as inline not quoting headers or inserting "original message" header
|
||
Comment 2•20 years ago
|
||
Was this fallout from bug 192557? Tomorrow's build should tell.
|
|
Reporter | |
Comment 3•20 years ago
|
||
> Was this fallout from bug 192557? Tomorrow's build should tell.
I've got neil's fix for bug 192557 and it doesn't fix this problem.
|
Assignee | |
Comment 4•20 years ago
|
||
Sorry, I can't reproduce this in my build, the original message table is fine.
|
||
Comment 5•20 years ago
|
||
I see this symptom in
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8a2) Gecko/20040710
For the plain cases -- text/plain, multipart/mixed with text/plain body, or
multipart/alternative displayed as plain -- forwarding is as expected, with the
separator, headers and message body and attachments.
HTML cases -- text/html, multipart/related, multipart/alternative displayed as
HTML, or multipart/mixed w/ text/html body -- when forwarded inline include only
the body (with attachments or embedded images).
Anyone seeing this under Linux or Mac? I assume there's no difference on those
platforms...
Keywords: regression
OS: Windows XP → Windows 2000
|
|
Assignee | |
Comment 6•20 years ago
|
||
The problem here is that mimedrft.cpp simply pastes the -- Original Message --
into the <HEAD> of the message, which is nonsense for HTML...
|
|
Assignee | |
Comment 7•20 years ago
|
||
Actually it pastes it before the <HEAD>, getting it ignored completely.
|
|
Assignee | |
Comment 8•20 years ago
|
||
|
|
Assignee | |
Updated•20 years ago
|
Assignee: sspitzer → neil.parkwaycc.co.uk
Status: NEW → ASSIGNED
|
|
Assignee | |
Updated•20 years ago
|
Attachment #152960 -
Flags: superreview?(bienvenu)
Attachment #152960 -
Flags: review?(mscott)
|
||
Updated•20 years ago
|
Attachment #152960 -
Flags: superreview?(bienvenu) → superreview+
|
||
Updated•20 years ago
|
Attachment #152960 -
Flags: review?(mscott) → review+
|
|
Assignee | |
Comment 9•20 years ago
|
||
Comment on attachment 152960 [details] [diff] [review]
Proposed patch
Trivial string text change to fix regression.
Attachment #152960 -
Flags: approval1.8a2?
|
|
Reporter | |
Comment 10•20 years ago
|
||
Comment on attachment 152960 [details] [diff] [review]
Proposed patch
a=sspitzer
Attachment #152960 -
Flags: approval1.8a2? → approval1.8a2+
|
|
Assignee | |
Comment 11•20 years ago
|
||
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
|
|
||
Comment 13•20 years ago
|
||
*** Bug 256028 has been marked as a duplicate of this bug. ***
|
||
Updated•20 years ago
|
Product: MailNews → Core
|
||
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•