950525 - Don't format HTML (inserting line break) if draft mode

Status: NEW

(MailNews Core :: Composition, defect)

Description

[Makoto Kato [:m_kato]]

When saving / sending mail, we use nsIDocumentEncoder with nsIDocumentEncoder::OutputFormatted.

nsIDocumentEncoder::OutputFormatted insert line break per wrap length.  So when composing mail then saving it as draft, line break may be unfortunately inserted.

So we should use nsIDocumentEncoder::OutputRaw when saving as draft.  When sending mail, nsMsgComposeAndSend is re-created, so we should use OutputFormatted with deliver mode only.

And force text mode, HTML text will be converted in nsMsgAttachmentHandler::UrlExit().  So we keep raw mode for format=flowed.

Comment 1

[Makoto Kato [:m_kato]]

Attachment: v1

Comment 2

[Thomas D. (:thomas8)]

Does this still happen? Current behaviour per comment 0 looks wrong, and proposal of comment 0 looks right! :)

Comment 3

[Thomas D. (:thomas8)]

Joshua, can you comment?