28493 - TXT-HTML Conversion for numbered lists omits initial spaces

Status: RESOLVED WORKSFORME

(SeaMonkey :: MailNews: Message Display, defect, P3)

Description

[Akkana Peck]

Send a plaintext message to yourself that looks something like this:

Here is a list:
     1. one
     2. two
     3. three

(I did this by making a numbered list in html compose, then having it convert to
plaintext on send.)

Now view this message in mozilla.  The spaces in front of the first list item
aren't shown: it looks like:
1. one
     2. two
     3. three

View it in 4.x or save it to disk to verify that the spaces are really there.

This is probably a layout issue, but it only happens in the mail window; if I
save to disk as test.txt and run the browser on that file, I see the spaces.

Comment 1

[Phil Peterson]

Reassign to rhp

Comment 2

[rhp (gone)]

Hi Akkana,
If you don't own this converter, please just reassign. Though the bug talks 
about a mail sending operation, it boils down to an issue with HTML - TXT 
conversion.

Thanks!

- rhp

Comment 3

[Akkana Peck]

Changing the summary -- it's apparently text-html conversion that's causing
problems, because the plaintext from the original html is fine when viewed in
4.x.

I own html->text (technically, anyway, though Ben and Daniel help a lot on it
and own some of the code) but I don't know much about text->html -- that's Ben's
area, isn't it?  Assigning back to Rich and cc'ing Ben.

Comment 4

[Ben Bucksch (:BenB)]

I have no idea, what in my code could cause this. It doesn't even know, if it's
the first or the second line.

Comment 5

[Ben Bucksch (:BenB)]

I can't reproduce it using a build from 2000-02-17 with format=flowed.

I can't test plain text, because Mozilla throws out all preferences I manually
add to prefs.js ("user_pref("mail.send_plaintext_flowed", false);" in this
case).

Comment 6

[rhp (gone)]

Will investigate further.

- rhp

Comment 7

[rhp (gone)]

Hi Daniel, I can recreate this issue pretty easily. Just compose an HTML message with a numbered list and then from the Options->Format menu, select "Plain Text Only". Now when you send it and receive it with Mozilla, you will see that the first line is over to the left column. I traced it into mozilla/mailnews/mime/src/mimetpfl.cpp and found that we get to the section in MimeInlineTextPlainFlowed_parse_line() where we are replacing single spaces with   but when we hit this first line, exdata->inflow has been set to TRUE so we don't do the replacement. I'd rather have you look at the problem since you understand this logic much better than I do. - rhp

Comment 8

[Daniel Bratell]

I can't reproduce this. When following the description rhp wrote I get:
BEGIN ------------- BEGIN
A list

   1. One
   2. Two
   3. Three
END ------------- END

It's like that in the Inbox file and on screen. 



      

Comment 9

[Daniel Bratell]

Note that there are no spaces at the end of the lines in the mail in the Inbox 
file except for the line after the list. rhp, could you please check if you have 
any spaces at the end of the lines in the raw mail.

Comment 10

[selmer (gone)]

Mass moving to M16 to get these off the M15 radar.  Please let me know if this
is really an M15 stopper.

Comment 11

[Daniel Bratell]

Since noone has commented, I'll mark this one WORKSFORME.