Open Bug 90257 Opened 23 years ago Updated 3 years ago

Several blanks are inserted in a quotation of list


(Core :: DOM: Serializers, defect, P5)





(Reporter: kazhik, Unassigned)


(Whiteboard: [have fix, need reviews])


(1 file, 2 obsolete files)

Several blanks are inserted in a quotation of list.

Steps to reproduce:

(1) Make a html mail which contains a list and save it as draft.
(2) Hit shift+Reply(or Reply) to reply the mail in plain text mode.

1. 1232. 456

This list is quoted as:

> 1.   123
> 2. 456
reassign to editor
Assignee: ducarroz → kin
Component: Composition → Editor: Core
Keywords: mailtrack
Product: MailNews → Browser
QA Contact: sheelar → sujay
There seems to be a difference in what the PlaintextSerializer writes out. When 
serializing is done based on an in memory DOM tree, the serializer 
itself drives the serializing process using recursive serializer calls to 
navigate the DOM tree, which seems to output the right thing.

In this particular case, the quoting code uses the Parser and NavDTD to control 
the serializing process, which seems to bypass some code used when serializing 
the in-memory DOM tree.
Assignee: kin → harishd
Component: Editor: Core → DOM to Text Conversion
--> peterv.
Assignee: harishd → peterv
Priority: -- → P3
Target Milestone: --- → mozilla1.1
This problem arises for the same reason as that for bug#107927. Patch for that 
bug fixes this too. 

*** This bug has been marked as a duplicate of 107927 ***
Closed: 22 years ago
Resolution: --- → DUPLICATE
Attached patch patch v0.1 (obsolete) — Splinter Review
The current code inserts a blank at the beginning of the list item.
What is that for? The quotation of list item works fine if I remove
that code.
I think this bug is nothing to do with bug 107927.
Resolution: DUPLICATE → ---
Attached patch patch v0.1 (obsolete) — Splinter Review
The previous attachment was wrong.
Attachment #123751 - Attachment is obsolete: true
I doubt that that's the right fix. Won't this break copying a word with a space
in front of it? After your patch we will just be dropping the space in front of
the selected word which is not what we want.
Attached patch patch v0.2Splinter Review
Thank you for the comment, peterv.
I updated the patch to output spaces when copying text from web page
to mail editor.
Attachment #123752 - Attachment is obsolete: true
Clearing milestone since 1.1 is long since gone. We have quite a bit of copy &
paste work lately, so I was wondering is this bug is still happening?
Whiteboard: [have fix, need reviews]
Target Milestone: mozilla1.1alpha → ---
Assignee: peterv → dom-to-text
Assignee: dom-to-text → nobody
QA Contact: sujay → dom-to-text

Bulk-downgrade of unassigned, >=5 years untouched DOM/Storage bugs' priority and severity.

If you have reason to believe this is wrong, please write a comment and ni :jstutte.

Severity: normal → S4
Priority: P3 → P5
You need to log in before you can comment on or make changes to this bug.