Closed
Bug 85150
Opened 23 years ago
Closed 22 years ago
HTML reply to plain text message with quoting contains spurious comment
Categories
(MailNews Core :: Composition, defect)
MailNews Core
Composition
Tracking
(Not tracked)
RESOLVED
WONTFIX
Future
People
(Reporter: neil, Assigned: bugzilla)
References
(Blocks 1 open bug)
Details
Using Build ID: 2001060704 Steps to reproduce problem: 1. Find a plain text message with quoted text. 2. Create a Reply to that message in HTML format. 3. Copy the message body. 4. Open a new editor window. 5. Paste the message in. 6. Switch to HTML source. Actual results: <pre><!----><br> in source. Expected results: No empty comment in source.
Assignee | ||
Updated•23 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → Future
Updated•22 years ago
|
Blocks: HTML-compose-tracker
Comment 1•22 years ago
|
||
I inserted that HTML comment specifically to prevent unforable layout (I think in Gecko, don't remember). There's a C++ comment in the C++ source code. WONTFIX.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
OS: Windows 95 → All
Hardware: PC → All
Resolution: --- → WONTFIX
Comment 2•22 years ago
|
||
s/unforable/unfavorable (bad)/
Reporter | ||
Comment 3•22 years ago
|
||
Just out of interest, where in the source code? I can't search for <!---->
Comment 4•22 years ago
|
||
mimetpla.cpp, I think.
Comment 5•22 years ago
|
||
<http://lxr.mozilla.org/seamonkey/source/mailnews/mime/src/mimetpla.cpp#387> 386 if (text->mCiteLevel == 0) 387 prefaceResultStr += "<!---->"; /* Make sure, NGLayout puts out 388 a linebreak */
Reporter | ||
Comment 6•22 years ago
|
||
I'm afraid I don't understand why it's only needed inside the outermost quote or in fact at all (I used Select All/Insert HTML to add and remove it and couldn't detect any difference). Perhaps it's to work around a subsequently fixed bug :-)
Comment 7•22 years ago
|
||
I didn't understand it either. It was just the way it was back then. Looked like a bug, but not sure. Yes, maybe it's fixed in the meantime, but maybe we want to preserve backwards compatibility with those older Mozillas, unless they are very old. I certainly don't care enough to investigate all that.
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
•