Closed Bug 23551 Opened 25 years ago Closed 25 years ago

[DOGFOOD] Mail compose quoted content is empty

Categories

(MailNews Core :: Composition, defect, P3)

Product:
MailNews Core
Component:
Composition
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: mcafee, Assigned: mozeditor)

Details

(Whiteboard: [PDT+]) 

linux,
mail compose auto-quote isn't working for me.
Severity: normal → critical
Summary: Mail compose quoting isn't happening for me → [DOGFOOD] Mail compose quoting isn't happening for me
Oh no!  Marking dogfood and raising severity.
Plain text, Chris?
plaintext.
MacAfee, could you be a little bit more descriptive of your problem?
Summary: [DOGFOOD] Mail compose quoting isn't happening for me → [DOGFOOD] Mail compose quoted content is empty
When I bring up mail compose window, I get

lchiang@netscape.com wrote:

.. and nothing else.  no message quoted.
Status: NEW → ASSIGNED
OS: Linux → All
Hardware: PC → All
Target Milestone: M13
Ok, I can reproduce the problem on Windows as well. It affect only plaintext
composition, HTML replies are fine.
Replying to plaintext mail is essential; marking PDT+.
Also -- ducarroz -- please set the milestone.
Assignee: ducarroz → akkana
Status: ASSIGNED → NEW
For some reason, the editor API InsertAsQuotation doesn't works anymore in
plaintext. Reassign to Akkana
Status: NEW → ASSIGNED
Accepting ... appears to be an editor problem.
Assignee: akkana → jfrancis
Status: ASSIGNED → NEW
In nsTextEditRules::WillDoAction(), info->action is 3008, kInsertElement,
because it wraps the inserted text in a <pre> node; and the text edit rules are
returning false.

Joe's last change to nsTextEditRules.cpp said it was for mail quoting, but none
of the changes seemed to be anything related to this.  I can't tell which change
broke this. But Joe is also the author of the code that calls the rules with
kInsertElement (which then isn't handled), so maybe he'll know more about this.

Joe, if you're overloaded and want me to fix this, please add a comment as to
how this was expected to work and assign the bug back to me.
QA Contact: lchiang → laurel
Status: NEW → ASSIGNED
accepting bug; probably something simple
Whiteboard: [PDT+]
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
ok, so we _are_ sometimes putting something other than text and breaks into
plaintext.  namely, pre.  I didnt know this.  I've altered the rules code to deal
with this.
Status: RESOLVED → VERIFIED
OK using:
2000-01-12-10 commercial NT 4.0
2000-01-12-08 commercial linux 6.0
2000-01-12-08 commercial mac OS 8.5.1
yes this looks fixed!  thanks.
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.