Open
Bug 90257
Opened 23 years ago
Updated 4 years ago
Several blanks are inserted in a quotation of list
Categories
(Core :: DOM: Serializers, defect, P5)
Tracking
()
NEW
People
(Reporter: kazhik, Unassigned)
Details
(Whiteboard: [have fix, need reviews])
Attachments
(1 file, 2 obsolete files)
|
(deleted),
patch
|
Details | Diff | Splinter Review |
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
|
||
Comment 1•23 years ago
|
||
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
|
||
Updated•23 years ago
|
Priority: -- → P3
Target Milestone: --- → mozilla1.1
|
||
Comment 4•23 years ago
|
||
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 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
|
Reporter | |
Comment 6•21 years ago
|
||
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.
|
|
Reporter | |
Comment 7•21 years ago
|
||
I think this bug is nothing to do with bug 107927.
Status: VERIFIED → REOPENED
Resolution: DUPLICATE → ---
|
|
Reporter | |
Comment 8•21 years ago
|
||
The previous attachment was wrong.
Attachment #123751 -
Attachment is obsolete: true
|
|
||
Comment 9•21 years ago
|
||
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.
|
|
Reporter | |
Comment 10•21 years ago
|
||
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 → ---
|
|
||
Updated•21 years ago
|
Assignee: peterv → dom-to-text
Status: REOPENED → NEW
|
||
Updated•15 years ago
|
Assignee: dom-to-text → nobody
QA Contact: sujay → dom-to-text
|
||
Comment 12•4 years ago
|
||
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.
Description
•