90257 - Several blanks are inserted in a quotation of list

Status: NEW

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

Description

[Koike Kazuhiko]

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

[Jean-Francois Ducarroz]

reassign to editor

Comment 2

[kinmoz]

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.

Comment 3

[harishd]

--> peterv.

Comment 4

[Tanu Mutreja]

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 ***

Comment 5

[sujay]

verified.

Comment 6

[Koike Kazuhiko]

Attachment: patch v0.1

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.

Comment 7

[Koike Kazuhiko]

I think this bug is nothing to do with bug 107927.

Comment 8

[Koike Kazuhiko]

Attachment: patch v0.1

The previous attachment was wrong.

Comment 9

[Peter Van der Beken [:peterv]]

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.

Comment 10

[Koike Kazuhiko]

Attachment: patch v0.2

Thank you for the comment, peterv.
I updated the patch to output spaces when copying text from web page
to mail editor.

Comment 11

[Heikki Toivonen (remove -bugzilla when emailing directly)]

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?

Comment 12

[Jens Stutte [:jstutte]]

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.