639196 - Rationalise popups into utilityOverlay.xul

Status: ASSIGNED

(SeaMonkey :: General, defect)

Description

[Ian Neal]

Attachment: Rationalise into utilityOverlay.xul patch v1.0

At the moment we have the following littered over various files in various places and they could be consolidated into a single popupset within utilityOverlay.xul:
<menupopup id="sidebarPopup"/>
<menupopup id="popupNotificationMenu"/>
<menupopup id="toolbar-context-menu"/>
<panel id="customizeToolbarSheetPopup"/>

Comment 1

[[e-mail]]

Comment on attachment 517165 [details] [diff] [review]
Rationalise into utilityOverlay.xul patch v1.0

<menupopup id="sidebarPopup"/>
Part of the sidebar, not really part of utilityOverlay. Not all windows have a sidebar (yet), only navigator, editor and compose. (And yet editor lacks the popup. Oops.)

<menupopup id="popupNotificationMenu"/>
Only needed in navigator and windows with a sidebar, since it handles the popup blocker notifications. I guess I only put it in utilityOverlay out of laziness.

<menupopup id="toolbar-context-menu"/>
<panel id="customizeToolbarSheetPopup"/>
These would be good candidates, but there are only two, which makes it seem less worthwhile. So I'm looking for feedback from Mnyromyr to see whether it's worth referencing all four popups in all windows.

Comment 2

[Karsten Düsterloh]

Comment on attachment 517165 [details] [diff] [review]
Rationalise into utilityOverlay.xul patch v1.0

Looking at <http://mxr.mozilla.org/comm-central/search?string=utilityOverlay.xul&find=suite%2F.*xul%24&findi=&filter=^[^\0]*%24&hitlimit=&tree=comm-central>, I'm not so sure that spreading the sidebar stuff to all those windows is that useful…

Comment 3

[Ian Neal]

(In reply to comment #2)
> Comment on attachment 517165 [details] [diff] [review]
> Rationalise into utilityOverlay.xul patch v1.0
> 
> Looking at
> <http://mxr.mozilla.org/comm-central/search?string=utilityOverlay.xul&find=suite%2F.*xul%24&findi=&filter=^[^\0]*%24&hitlimit=&tree=comm-central>,
> I'm not so sure that spreading the sidebar stuff to all those windows is that
> useful…

Surely it would only spread stuff into windows that have the hook for popupset utilityOverlaySet in?

Comment 4

[Philip Chee]

IanN are you still working on this?