633910 - Popup menus clipped by window - should use separate windows

Status: RESOLVED INCOMPLETE

(Core Graveyard :: Widget: Qt, defect)

Description

[Florian Hänel [:heeen]]

Right now for a Desktop build, menus are clipped by the main window, as they are drawn by xulrunner directly on top of the main xul view.
We should create separate Qt windows for each popup menu to faciliate menus bigger than the main window, drop shadows and other platform window decorations on popup and context menus.

Comment 1

[Florian Hänel [:heeen]]

Attachment: work in progress

Here's a first patch that pushes menus in their own widget. There are still a few problems, though:
* I couldn't use the Qt::Popup window flag because the QGraphicsView will eat all events and never release its grab on the keyboard and mouse - very bad
* not using Qt::Popup causes all menus to appear as a taskbar item
* sometimes the content on menus will vanish if you open a second menu or hover around a bit.

Comment 2

[Oleg Romashin (:romaxa)]

I think this bug would be interesting for Tatiana, because she was fixing some problems around this area

Comment 3

[Florian Hänel [:heeen]]

Attachment: hovering works, proper popup flag

Latest WIP patch. This uses Qt::Popup which should prevent the popup menus from getting thier own taskbar entry. I had to write logic to forward mouse events to parent menus and the main window because of the modality that Qt::Popup causes.

Comment 4

[Benjamin Smedberg]

In bug 1282866, I have removed the QT code from the Firefox tree with the approval of the active peer. That code is not currently maintained by any team.

I have looked through the bugs in the Widget: Qt bugzilla component and I believe all of these are no longer relevant because the Qt code has been removed. If you believe that this bug is still valid, please move it to another more appropriate bugzilla component as you reopen it.