Closed Bug 361161 Opened 13 years ago Closed 13 years ago

Reporter doesn't work for suiterunner (MOZ_XUL_APP=1) builds

Categories

(Other Applications Graveyard :: Reporter, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

Report Broken Web Site doesn't appear in the help menus for SeaMonkey builds where MOZ_XUL_APP=1. This will need fixing for when SeaMonkey becomes an xul app.

I suspect the problem is with http://lxr.mozilla.org/seamonkey/source/extensions/reporter/jar.mn

I'll try and get a patch done in a few days if no one else does.
I have a patch for this, taking.
Assignee: robert → bugzilla
As SeaMonkey & Firefox are the only ones that build reporter, we can simply enable all the overlays, and add another one to overlay navigator.xul as required by SeaMonkey. Neil has assured me this addition won't affect ff.
Attachment #245961 - Flags: review?(robert)
Comment on attachment 245961 [details] [diff] [review]
Fixes the problem

Didn't know you can do that.  Provided you've made sure this is ok on Firefox and SeaMonkey

r=me
Attachment #245961 - Flags: review?(robert) → review+
Robert: This can be done is SeaMonkey just like in Firefox once we build SeaMonkey as a MOZ_XUL_APP, which this bug is all about. Our plan is to switch to that on trunk in the long term, and we are heavily working on getting everything to perform as expected there :)
I checked on Firefox and this doesn't affect it, and it fixes suiterunner builds of SeaMonkey.

checked in:

Checking in extensions/reporter/jar.mn;
/cvsroot/mozilla/extensions/reporter/jar.mn,v  <--  jar.mn
new revision: 1.8; previous revision: 1.7
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Product: Other Applications → Other Applications Graveyard
You need to log in before you can comment on or make changes to this bug.