29369 - Conditionally use variable width fonts for msg display

Status: VERIFIED FIXED

(MailNews Core :: MIME, defect, P3)

Description

[Ben Bucksch (:BenB)]

This bug consists of 2 parts (separate in 2 bugs, if you want):
1. Flowed text (format=flowed) should display in variable width font.
2. "Display plain text msgs ... with | variable width font" should be honored.

For part 1, I don't know about preformatted parts. Daniel?

Comment 1

[rhp (gone)]

I really don't think we are looking for more UI in the product at this point. 
Also, I'm not sure I understand #1. I currently do see format=flowed messages 
displayed in variable width fonts?

- rhp

Comment 2

[Ben Bucksch (:BenB)]

rhp,
1. In my build from 2000-02-25 (PST), I see format=flowed displayed in a font,
which looks like Courier 14pt.
2. The pref UI I mentioned exists, it just has no effect.

Comment 3

[rhp (gone)]

Ok, the first part of this bug seems to be working fine for me on Win32 and
Linux. I am investigating the second part about the:

"Display plain text msgs ... with | variable width font" should be honored.

Comment 4

[rhp (gone)]

Ok, found the problem with the second part of this. Have a fix in my tree, but 
there is a problem saving the pref now that I emailed seth about.

- rhp

Comment 5

[rhp (gone)]

Here is the pref problem:
I think there is a problem saving the pref: 

                    mail.fixed_width_messages 

If I hard code it in the prefs to true, then the UI shows it correctly and I 
can set it to use "variable width font"s, but the problem is trying
to go back to fixed width. It never seems to take. 

I noticed this because I am fixing a problem with the feature itself and 
noticed this behavior. 

- rhp

Comment 6

[Ben Bucksch (:BenB)]

> Ok, found the problem with the second part

Cool! If the fix is not too diffcult, I volunteer as reviewer.

> Ok, the first part of this bug seems to be working fine for me

I found my problem: my x-western seif was sat to helvetica, but x-unicode serif
was still sat to charter 16pt.

Comment 7

[rhp (gone)]

The old tags we used in 4.x for tweaking display fonts for <PRE> parts must have
been non-standard. Fixed now.

- rhp

Comment 8

[pmock]

changing qa assigned to myself.

Comment 9

[pmock]

Verified as fixed on win32, linux, and macos using the following builds:
 win32 commercial seamonkey build 2000-121112-mtrunk installed on P500 Win98
 linux commercial seamonkey build 2000-121112-mtrunk installed on P200 RedHat 
6.2
 macos commercial seamonkey build 2000-121112-mtrunk installed on G3/400 OS 9.04