108478 - Some pref panel menulists not set to default value

Status: VERIFIED FIXED

(SeaMonkey :: Preferences, defect)

Description

[John Morrison]

blake. I noticed that the menulists in these files:

  mailnews/base/prefs/resources/content/pref-viewing_messages.xul
  mailnews/compose/prefs/resources/content/pref-composing_messages.xul
  xpfe/components/prefwindow/resources/content/pref-languages.xul

were not getting set to a default value when the panel loads. I see that 
you removed the preftype for these menulists, but that preftype does not 
have a default type (preftype) in nsPrefWindow.js. I don't know whether 
menulist needs to get a default, or whether the preftype is required if 
a menulist (but guessing the latter).

Comment 1

[Blake Ross]

John, great catch. It should have a default of int. I could have sworn I did 
that.

Comment 2

[John Morrison]

did you also need this same change at the attached URL (earlier in savePrefs)

Comment 3

[sairuh (rarely reading bugmail)]

vrfy fixed --except that the menulist in Languages isn't initialized; see bug
108743 for that. used commercial bits:

linux [rh6.2] - 2001.11.06.12
winNT - 2001.11.06.12
mac os 10.1 - 2001.11.06.08

Comment 4

[[e-mail] (away - not reading bugmail)]

>menulist in Languages isn't initialized

this should work with 1107 bits