Name of Default HTML Message Font not shown in Preferences

RESOLVED FIXED in seamonkey2.15

Status

SeaMonkey
Preferences
RESOLVED FIXED
8 years ago
5 years ago

People

(Reporter: Andy Boze, Assigned: Philip Chee)

Tracking

Trunk
seamonkey2.15

SeaMonkey Tracking Flags

(seamonkey2.15 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
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.

Updated

8 years ago
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
(Assignee)

Comment 1

5 years ago
Created attachment 660054 [details] [diff] [review]
Patch v1.0 Proposed fix.

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)

Updated

5 years ago
Assignee: philip.chee → nobody
Severity: minor → normal
Component: MailNews: General → Preferences
OS: Windows XP → All
Hardware: x86 → All
(Assignee)

Updated

5 years ago
Assignee: nobody → philip.chee

Comment 2

5 years ago
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+
(Assignee)

Comment 3

5 years ago
Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/082c976e2c8c
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
status-seamonkey2.15: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.15
You need to log in before you can comment on or make changes to this bug.