[Merge] lp:~albaguirre/qtubuntu/use-mir-surface-apis into lp:qtubuntu
Alberto Aguirre
alberto.aguirre at canonical.com
Mon Aug 10 18:13:13 UTC 2015
Review: Needs Fixing
Actually this looks like it will need some more work. After the rebase and mir 0.14 things are not working quite right...
There's some issues:
- Displaying a menu (http://pastebin.ubuntu.com/12049608/)
-- No rendering is done (looks grey) until the mouse hovers the menu area - adding a handleExposeEvent during setGeometry helps but I don't think that's the right place for it.
-- Qt destroys the menu window immediately after shown - mir sends a focus false event for the parent surface and focus true event for the menu surface which is translated to handleWindowActivated(0, Qt::ActiveWindowFocusReason) and handleWindowActivated(menu_surface, Qt::ActiveWindowFocusReason).
Since the menu surface is already visible the first call makes Qt destroy the menu surface.
-- mir is not destroying submenu surfaces when they lose focus
Visibility and window state are both linked to the mir surface state - if a window is made invisible, this makes the mir surface always stay in the minimized state.
--
https://code.launchpad.net/~albaguirre/qtubuntu/use-mir-surface-apis/+merge/267228
Your team Ubuntu Phablet Team is subscribed to branch lp:qtubuntu.
More information about the Ubuntu-reviews
mailing list