nsbr in HTML tags put out for flowed msgs

VERIFIED FIXED in M17

Status

MailNews Core
MIME
P3
normal
VERIFIED FIXED
18 years ago
9 years ago

People

(Reporter: BenB, Assigned: BenB)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: r=rhp, a=waterson)

(Assignee)

Description

18 years ago
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)

Comment 1

18 years ago
/
Assignee: rhp → mozilla
(Assignee)

Comment 2

18 years ago
Hm, WORKSFORME with 2000-06-22 nightly. But there is a bug in the code, I think
(and Daniel does, too).
(Assignee)

Updated

18 years ago
Summary: nsbr in HTML tags put out → nsbr in HTML tags put out for flowed msgs
(Assignee)

Comment 3

18 years ago
> 4. Send

as plain text only
(Assignee)

Comment 4

18 years ago
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
(Assignee)

Updated

18 years ago
Keywords: patch
(Assignee)

Updated

18 years ago
Keywords: approval
Whiteboard: Fixed. Patch in bug 40862. → Fixed. Patch in bug 40862.Fixed. Waiting for bug 40862's approval
(Assignee)

Comment 5

18 years ago
Fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
Whiteboard: Fixed. Patch in bug 40862.Fixed. Waiting for bug 40862's approval
(Assignee)

Updated

18 years ago
Whiteboard: r=rhp, a=waterson

Comment 6

17 years ago
Fix not in 9/29 build. Reopening...
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 7

17 years ago
Ryan, can you comment, please? Worksforme.
(Assignee)

Comment 8

17 years ago
No reaction, closing again to force one.
Status: REOPENED → RESOLVED
Last Resolved: 18 years ago17 years ago
Resolution: --- → FIXED

Comment 9

17 years ago
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.