Closed Bug 43453 Opened 24 years ago Closed 24 years ago

nsbr in HTML tags put out for flowed msgs

Categories

(MailNews Core :: MIME, defect, P3)

Product:
MailNews Core
Component:
MIME
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: BenB, Assigned: BenB)

Details

(Whiteboard: r=rhp, a=waterson) 

Reproduction (guessed from code):
1. Create a new mail
2. Write a long ( > 1 line) paragraph without hard linebreaks
3. The paragraph should contain " *foo* "
4. Send
5. Read

Actual result:
Not the expected result :).

Expected result:
"*foo*" is shown in bold.

Additional comments:
"intag" prtection if missing from the flowed code in the flowed class. I added
it.

Additional, related bug: intag is never stopped, i.e. if there's an HTM tag
found, i.e. spaces in the rest of the line are collapsed. I don't know, if I
want to fix that.
/
Assignee: rhp → mozilla
Hm, WORKSFORME with 2000-06-22 nightly. But there is a bug in the code, I think
(and Daniel does, too).
Summary: nsbr in HTML tags put out → nsbr in HTML tags put out for flowed msgs
> 4. Send

as plain text only
Fixed. Patch in bug 40852, which touches the same part.

I didn't fix the "never-stopping |intag|-mode" bug. I want to leave some fun for
Daniel :->. Feel free to file a bug on it, cc me.
Status: NEW → ASSIGNED
Whiteboard: Fixed. Patch in bug 40862.
Target Milestone: --- → M17
Keywords: patch
Keywords: approval
Whiteboard: Fixed. Patch in bug 40862. → Fixed. Patch in bug 40862.Fixed. Waiting for bug 40862's approval
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Whiteboard: Fixed. Patch in bug 40862.Fixed. Waiting for bug 40862's approval
Whiteboard: r=rhp, a=waterson
Fix not in 9/29 build. Reopening...
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Ryan, can you comment, please? Worksforme.
No reaction, closing again to force one.
Status: REOPENED → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → FIXED
oops! ...I misread the bug! Everything is working.

Verified in 10/5 build.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.