5476 - Preference radio buttons are not initialized

Status: VERIFIED FIXED

(SeaMonkey :: Preferences, defect, P2)

Description

[Stephen P. Morse]

Open the preference panel (edit/preferences) and go to advanced.  There are
three radio buttons for cookie behavior -- namely accept all, accept only, and
disable.  The default should be accept all.  However, none of them are checked.

And a related issue -- the wording on "accept only" is bad.  It should be
"accept only cookies that get sent back to originating server"

Comment 1

[don]

Assigned to matt.

Is this fixable for M5?

Comment 2

[matt]

done

Comment 3

[leger]

Updating QA Contact from paulmac to cpratt

Comment 4

[cpratt]

yup, it's fixed. 1999052408 build. NT 4.

Comment 5

[Stephen P. Morse]

No, this is not fixed in the general case.  Yes, it is true that the specific
example cited here (cookie preferences) has been fixed.  But none of the other
radio buttons on preferences have been initialized.  For example, the startup
preference panel (appearance) has a radio button for "show toolbars as" and
initially nothing is checked there.  Similarly under Mail-and-news/addressing
there is an unititialized radio button for "when there are multiple addresses
found".  And in offline, there are two uninitialized radio buttons -- namely
"startup state" and "when going offline".  And another uninitialized one in
advanced/cache.

Finally, there are two other problems with the preference panel that I am
dumping into this report because I am too lazy to open another one.  Namely:

- Navigator/Languages does not open a new panel
- Panels that are large get truncated on the bottom and there is no scroll
button.  See mail-and-news/identity as an example.

Comment 6

[matt]

We are changing the prefs into xul prefs
so i am not working on the old prefs that are in
the product now.  When the xul prefs are changed into
the product it will be time to review the prefs.
Also the languages prefs are not put in yet because
we don't have a proper widget for it and the scrollbars
are off is a feature not a bug.  The mail people have
to change there prefs pane.

Comment 7

[leger]

Clearing Fixed resolution.

Comment 8

[Michael La Guardia]

adding bug to the Prefs feature tracking dependency list

Comment 9

[matt]

On the browser prefs this seems to happen only to offline now

Comment 10

[matt]

since we are not using offline i'm mvoing this out

Comment 11

[matt]

the offline prefs are now default.
Mail news ones are not in all.js or andy .js

Comment 12

[sairuh (rarely reading bugmail)]

spam: in my testing realm, so reassigning qa contact to me, en masse.

Comment 13

[chuang]

Mailnews default is in mailnew.js.  Remove the "checked = true" from .xul file
to reflect the default.  Fix checked in tonight.

Comment 14

[sairuh (rarely reading bugmail)]

okay, for the most part all radio buttons in the prefs have initial values...
except in a few places:

IM > Privacy
Composer
Offline > Download

i'll open separate bugs for these!

Comment 15

[leger]

Bulk move of all Pref UI component bugs to new Preferences component.  Pref UI 
component will be deleted.