Closed Bug 288969 Opened 20 years ago Closed 20 years ago

Fix overlay madness

Categories

(Other Applications Graveyard :: Reporter, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: raccettura, Assigned: raccettura)

Details

Attachments

(1 file, 1 obsolete file)

Overlay's look like a game of "pick up sticks" (for those kids who remember
non-electronic games).

Fix coming per Neil and mconnor suggestions.
Attached patch Patch (obsolete) — Splinter Review
Attachment #179584 - Flags: review?(neil.parkwaycc.co.uk)
Comment on attachment 179584 [details] [diff] [review]
Patch

baseMenuOverlay (FF) and utilityOverlay (Suite) will overlay bookmarks, DOM
Inspector etc. You should only overlay the browser/navigator.
Attachment #179584 - Flags: review?(neil.parkwaycc.co.uk) → review-
Attached patch Patch v2Splinter Review
Attachment #179626 - Flags: review?(neil.parkwaycc.co.uk)
Attachment #179584 - Attachment is obsolete: true
Comment on attachment 179626 [details] [diff] [review]
Patch v2

But please please please overlay the navigator instead of utilityOverlay before
you check in.
Attachment #179626 - Flags: review?(neil.parkwaycc.co.uk) → review+
Checking in extensions/reporter/resources/content/reporter/contents.rdf;
/cvsroot/mozilla/extensions/reporter/resources/content/reporter/contents.rdf,v 
<--  contents.rdf
new revision: 1.3; previous revision: 1.2
done
Checking in extensions/reporter/resources/content/reporter/reporterOverlay.xul;
/cvsroot/mozilla/extensions/reporter/resources/content/reporter/reporterOverlay.xul,v
 <--  reporterOverlay.xul
new revision: 1.3; previous revision: 1.2
done


Should be totally fixed now.  Thanks for the help Neil.
Status: NEW → RESOLVED
Closed: 20 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.

Attachment

General

Created:
Updated:
Size: