Neil Deakin Assignee
	Description • 17 years ago

The widget code currently expects all popups to behave as menus. However, this isn't actually the case and we have several panel popups, and more have been added recently (identity popup, places popup, customize dialog, etc). Panels should behave as toplevel windows, have no window decorations, may optionally roll up (gRollupWidget will be null if they don't), and receieve key and mouse events.
For instance, one issue is that popups are not made active or focused, yet panels should be focused as regular windows.

There are numerous bugs caused by this or that are harder to fix due to this.

	Neil Deakin Assignee
	Comment 1 • 17 years ago

This patch doesn't really fix the bug, but improves some problems with panels and focus behaviour in popups. It fixes bugs 395123, 395334 and 390178, and bug 124313. It changes the popup to have a parent so that the child popup is associated with the parent, moves with the parent and appears just over the parent but under other windows. It implements the 'dependent' window open flag on Mac. It also changes noautohide="true" panels so that they can only be used in chrome windows.

	Asaf Romano (gone)
	Comment 2 • 17 years ago

Neil: With this patch, the customization sheet (which is a noauthohide-panel) ignores all mouse events.

	Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)
	Comment 3 • 17 years ago

-'ing for now, but please argue for a + in the 1.9 meeting for this.

	Neil Deakin Assignee
	Comment 4 • 17 years ago

This patch fixes the bugs mentioned above, as well as 400019 and 385609 in certain situations.

	Neil Deakin Assignee
	Comment 5 • 16 years ago

Much of the stuff in these patches has already been checked in, or is part of other bugs. Going to just close this bug off.