Closed
Bug 14663
Opened 25 years ago
Closed 25 years ago
JavaScript Error: ReferenceError: pref is not defined
Categories
(SeaMonkey :: Preferences, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
M11
People
(Reporter: norrisboyd, Assigned: mcafee)
Details
(Whiteboard: Fix in hand, r=matt)
When selecting Edit/Preferences... the message "JavaScript Error: ReferenceError: pref is not defined" appears on the console. The error appears to be line 30 in pref tree.xul: onload="ChangePanelURL(pref-appearance.xul);" Should that be onload="ChangePanelURL('pref-appearance.xul');" (note the addition of single quotes). After making the change I get an an assertion when ChangePanelURL is called that "panel window is null".
Assignee | ||
Updated•25 years ago
|
Assignee: shuang → mcafee
Assignee | ||
Updated•25 years ago
|
Target Milestone: M11
Assignee | ||
Comment 1•25 years ago
|
||
stealing this from shuang.
Assignee | ||
Comment 2•25 years ago
|
||
mozilla/xpfe/components/prefwindow/resources/content/preftree.xul
Assignee | ||
Comment 3•25 years ago
|
||
fixed. Will open another bug on the assertion problem.
Assignee | ||
Comment 4•25 years ago
|
||
this never got fixed! Anyways, "reopening".
Assignee | ||
Comment 5•25 years ago
|
||
Still happening, I think we should put the quotes back and get rid of the assertion, not sure why that's there. We must be blowing off some initialization some how, but ? maybe just returning for that case is Ok.
Assignee | ||
Updated•25 years ago
|
Whiteboard: Fix in hand, r=matt
Assignee | ||
Comment 6•25 years ago
|
||
Fix in hand, r=matt
Assignee | ||
Updated•25 years ago
|
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 7•25 years ago
|
||
Fixed. No JS error when bringing up prefs window now. r=matt.
Using the 1999111016 build under NT, I don't see an error in the console. Marking verified.
Bulk move of all Pref UI component bugs to new Preferences component. Pref UI component will be deleted.
Component: Pref UI → Preferences
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•