[FIXr]Scripts panel scripts broken

RESOLVED FIXED in mozilla1.6final

Status

SeaMonkey
Preferences
P1
major
RESOLVED FIXED
15 years ago
14 years ago

People

(Reporter: bz, Assigned: bz)

Tracking

Trunk
mozilla1.6final
x86
Linux

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

From bug 227612:

Disabling JS for Navigator didn´t gray out the JS-settings

"That got broken by the patch to bug 181973
-- the Startup() function for that pref panel has a JS error."
Created attachment 136919 [details] [diff] [review]
Patch

The key here is the removal of the stray '}'
Attachment #136919 - Flags: superreview?(alecf)
Attachment #136919 - Flags: review?(neil.parkwaycc.co.uk)
Priority: -- → P1
Summary: Scripts panel scripts broken → [FIX]Scripts panel scripts broken
Target Milestone: --- → mozilla1.6final

Comment 2

15 years ago
Comment on attachment 136919 [details] [diff] [review]
Patch

Doh :-[
Attachment #136919 - Flags: review?(neil.parkwaycc.co.uk) → review+
Would be good to not ship 1.6b with this...
Flags: blocking1.6b?

Comment 4

15 years ago
Add semicolon after
document.getElementById("pluginPreferences")

Comment 5

15 years ago
Comment on attachment 136919 [details] [diff] [review]
Patch

sr=alecf
the semicolon would be nice, to not confuse emacs
Attachment #136919 - Flags: superreview?(alecf) → superreview+
Comment on attachment 136919 [details] [diff] [review]
Patch

Could this please be approved for 1.6b?  Simple removal of stray '}' char that
makes this panel actually work...
Attachment #136919 - Flags: approval1.6b?
Summary: [FIX]Scripts panel scripts broken → [FIXr]Scripts panel scripts broken
Attachment #136919 - Flags: approval1.6b? → approval1.6b+
Checked in for 1.6b.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
For the record:
Comment 7 check in includes both the patch and comment 4 suggestion :-)
Flags: blocking1.6b?
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.