Closed
Bug 44363
Opened 24 years ago
Closed 24 years ago
Put out linebreaks in HTML for format=flowed
Categories
(MailNews Core :: MIME, defect, P4)
MailNews Core
MIME
Tracking
(Not tracked)
VERIFIED
WORKSFORME
M18
People
(Reporter: BenB, Assigned: BenB)
Details
(Whiteboard: [NEED INFO] Trivial once I know which linebreak to put out)
With some of my recent changes changes, we currently put out one lone HTML line for format=flowed. Not very readable. We should insert a linebreak e.g. after each source line. Question: That linebreak exactly? |'\n'|? |NS_LINEBREAK|?
Assignee | ||
Updated•24 years ago
|
Priority: P3 → P4
Target Milestone: --- → M18
Assignee | ||
Updated•24 years ago
|
Status: NEW → ASSIGNED
Whiteboard: Trivial once I know which linebreak to put out
Assignee | ||
Updated•24 years ago
|
Summary: Put out linebreaks for format=flowed → Put out linebreaks in HTML for format=flowed
Assignee | ||
Updated•24 years ago
|
Whiteboard: Trivial once I know which linebreak to put out → [NEED INFO] Trivial once I know which linebreak to put out
Assignee | ||
Updated•24 years ago
|
Severity: normal → minor
Assignee | ||
Updated•24 years ago
|
Target Milestone: M18 → M17
Assignee | ||
Updated•24 years ago
|
Target Milestone: M17 → M18
Comment 1•24 years ago
|
||
If it's for the layout engine it really doesn't matter. I'm not sure what NS_LINEBREAK contains but \n is just one byte so you could save one byte per line break *pause for applause* :-).
Assignee | ||
Comment 2•24 years ago
|
||
Daniel, the output is also used for quotes (which we later send out, if sent as HTML) and SaveAs. That's why it matters. Some people read the HTML source of msgs they recieve.
Comment 3•24 years ago
|
||
But that's after the data has passed some composer window and then been converted to HTML (from XIF or what it's called). But why don't you just check what happens? In the mail it should be CRLF and I think it will be that no matter what you end lines with in the mime classes.
Assignee | ||
Comment 4•24 years ago
|
||
> But that's after the data has passed some composer window
Not for SaveAs. Rich and akk said something about that running through the
"output system", but I don't know what that is. I'll just try out their
recommondations, when I have time.
Assignee | ||
Comment 5•24 years ago
|
||
I don't know why, but it seems to be fixed. WORKSFORME.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → WORKSFORME
Verified as worksforme on win32, linux, and macos using the following builds: win32 commercial seamonkey build 2000-091906-m18 installed on P500 Win98 linux commercial seamonkey build 2000-091906-m18 installed on P200 RedHat 6.2 macos commercial seamonkey build 2000-091904-m18 installed on G3/400 OS 9.04
Status: RESOLVED → VERIFIED
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
•