Closed Bug 529418 Opened 11 years ago Closed 8 years ago

Name of Default HTML Message Font not shown in Preferences

Categories

(SeaMonkey :: Preferences, defect)

defect
Not set

Tracking

(seamonkey2.15 fixed)

RESOLVED FIXED
seamonkey2.15
Tracking Status
seamonkey2.15 --- fixed

People

(Reporter: base12, Assigned: philip.chee)

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.6pre) Gecko/20091117 SeaMonkey/2.0.1pre Firefox/3.5.4
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.6pre) Gecko/20091117 SeaMonkey/2.0.1pre

In the Mail & Newsgroups Preferences (not the Account Preferences), you can set a default font for HTML messages. Although you can successfully set a font, if you close Preferences window, then reopen it, the name of the selected font will not be displayed, just a blank box. The exceptions to this are the five font types at the top of the list: Variable Width; Fixed Width; Helvetica, Arial; Times; and Courier. If any of those five are selected, they will be displayed in the font selection box.

Reproducible: Always

Steps to Reproduce:
1. Open Edit | Preferences | Mail & Newsgroups | Composition
2. In the "Defaults for HTML Messages" section, select any font other than one of the first five.
3. Click OK to close the Preferences window.
4. Repeat step 1.
Actual Results:  
The name of the previously selected font is not shown. The box is blank.

Expected Results:  
The font name should be shown.

The selected font is actually saved in preferences, as can be shown by attempting to compose an HTML e-mail. The name of the font is just not shown in the Preferences window. Also reported to happen in SM 2.0.
Severity: normal → minor
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: Default HTML message font not displaying in Preferences → Name of Default HTML Message Font not shown in Preferences
Version: unspecified → Trunk
Based on Thunderbird Bug 468774 (Preference window incorrectly shows default font).

> -      if (localFonts[i] != "") {
Is this ever empty?
Assignee: nobody → philip.chee
Status: NEW → ASSIGNED
Attachment #660054 - Flags: review?(iann_bugzilla)
Assignee: philip.chee → nobody
Severity: minor → normal
Component: MailNews: General → Preferences
OS: Windows XP → All
Hardware: x86 → All
Assignee: nobody → philip.chee
Comment on attachment 660054 [details] [diff] [review]
Patch v1.0 Proposed fix.

>     var localFontCount = { value: 0 }
>     var localFonts = enumerator.EnumerateAllFonts(localFontCount);
We don't use localFontCount any more so could just do enumerator.EnumerateAllFonts({});

Toolkit doesn't check for != "" whereas SM and TB do. Neil might know the answer.
Attachment #660054 - Flags: review?(iann_bugzilla) → review+
Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/082c976e2c8c
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.15
You need to log in before you can comment on or make changes to this bug.