(a) is my suggestion, though I could see (b) as an option too. If you don't need the text zoom code, try to avoid it (and its JS!) getting overlaid. this goes for most of the overlays. The xul itself won't hurt too much, because whatever can't get overlayed won't get built, but the JS will always be there taking space. This might mean moving the viewZoomOverlay.xul out of mailWindowOverlay and moving it into the mail xul files (below the mailWindowOverlay include) that actually do want this menu. Or it might mean refactoring mailWindowOverlay. I'll leave that up to you guys.
Last time I looked Search only needed the status bar XUL from mailWindowOverlay
Created attachment 108737 [details] [diff] [review] Proposed Patch Removes mailWindowOverlay and inlines the required elements.
I'm a little leary of going this direction as the search window takes on more of the attributes of a real window instead of less, but as long as you've made sure everything still works, I can sr it.
Comment on attachment 108737 [details] [diff] [review] Proposed Patch sr=bienvenu - we still include mailOverlay.js, so you won't get any savings there...
Attachment #108737 - Flags: superreview?(bienvenu) → superreview+
Comment on attachment 108737 [details] [diff] [review] Proposed Patch r=jag
Attachment #108737 - Flags: review?(jaggernaut) → review+
Assignee: naving → neil
Oops, timeless checked this in ages ago :-)
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
Component: MailNews: Search → MailNews: Message Display
QA Contact: laurel → search
You need to log in before you can comment on or make changes to this bug.