Closed Bug 308966 Opened 19 years ago Closed 19 years ago

Warn checkbox not enabling/disabling in update options

Categories

(Firefox :: Settings UI, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: mossop, Assigned: mossop)

Details

(Keywords: fixed1.8)

Attachments

(1 file, 1 obsolete file)

When the options window is not in instant apply, the warn checkbox doesnt enable and disable with the radio button setting until the options window is closed and opened again. I have a fix in progress.
Status: NEW → ASSIGNED
Flags: blocking1.8b5?
Attached patch Patch (obsolete) — Splinter Review
A change event is necessary so the onchange handler on the preference is called. The call to updateElements is then unnecessary since that will happen in the change handler in the preference binding.
Attachment #196433 - Flags: review?(mconnor)
Attached patch Second patchSplinter Review
On second thoughts. It's only sensible to fire the change event when there is an actual change, not the situation when we set the preference's value at the start. There we should only update the elements.
Attachment #196433 - Attachment is obsolete: true
Attachment #196434 - Flags: review?(mconnor)
Attachment #196433 - Flags: review?(mconnor)
Attachment #196434 - Flags: review?(mconnor) → review+
Fixed on trunk
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Attachment #196434 - Flags: approval1.8b5?
Attachment #196434 - Flags: approval1.8b5? → approval1.8b5+
Flags: blocking1.8b5? → blocking1.8b5+
Checking in preferences.xml; /cvsroot/mozilla/toolkit/content/widgets/preferences.xml,v <-- preferences.xml new revision: 1.31.2.4; previous revision: 1.31.2.3 done
Keywords: fixed1.8
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: