Closed
      
        Bug 288969
      
      
        Opened 20 years ago
          Closed 20 years ago
      
        
    
  
Fix overlay madness
Categories
(Other Applications Graveyard :: Reporter, defect)
Tracking
(Not tracked)
        RESOLVED
        FIXED
        
    
  
People
(Reporter: raccettura, Assigned: raccettura)
Details
Attachments
(1 file, 1 obsolete file)
| 3.07 KB,
          patch         | neil
:
              
              review+ | Details | Diff | Splinter Review | 
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.
| Assignee | ||
| Comment 1•20 years ago
           | ||
        Attachment #179584 -
        Flags: review?(neil.parkwaycc.co.uk)
| Comment 2•20 years ago
           | ||
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-
| Assignee | ||
| Comment 3•20 years ago
           | ||
        Attachment #179626 -
        Flags: review?(neil.parkwaycc.co.uk)
| Assignee | ||
| Updated•20 years ago
           | 
        Attachment #179584 -
        Attachment is obsolete: true
| Comment 4•20 years ago
           | ||
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+
| Assignee | ||
| Comment 5•20 years ago
           | ||
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
| Updated•7 years ago
           | 
Product: Other Applications → Other Applications Graveyard
          You need to log in
          before you can comment on or make changes to this bug.
        
Description
•