Closed
Bug 308966
Opened 19 years ago
Closed 19 years ago
Warn checkbox not enabling/disabling in update options
Categories
(Firefox :: Settings UI, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: mossop, Assigned: mossop)
Details
(Keywords: fixed1.8)
Attachments
(1 file, 1 obsolete file)
1.32 KB,
patch
|
mconnor
:
review+
asa
:
approval1.8b5+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Updated•19 years ago
|
Status: NEW → ASSIGNED
Flags: blocking1.8b5?
Assignee | ||
Comment 1•19 years ago
|
||
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)
Assignee | ||
Comment 2•19 years ago
|
||
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)
Assignee | ||
Updated•19 years ago
|
Attachment #196433 -
Flags: review?(mconnor)
Updated•19 years ago
|
Attachment #196434 -
Flags: review?(mconnor) → review+
Assignee | ||
Comment 3•19 years ago
|
||
Fixed on trunk
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•19 years ago
|
Attachment #196434 -
Flags: approval1.8b5?
Updated•19 years ago
|
Attachment #196434 -
Flags: approval1.8b5? → approval1.8b5+
Updated•19 years ago
|
Flags: blocking1.8b5? → blocking1.8b5+
Comment 4•19 years ago
|
||
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.
Description
•