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: