Comment 6 for bug 1380702

Revision history for this message
Ɓukasz Zemczak (sil2100) wrote :

Ok, identified the initial problem here. It's not any recent problem - it's an issue that seems to be with appmenu-qt5 and Qt5 since the very beginning. It's not really a bug in appmenu-qt5, more like a design decision that Qt5 had that I wasn't aware of.

The main problem here is that once we load up the QPA platformtheme and the QMenuBar object is able to create the QPlatformMenuBar instead, what it does is it hides the original QMenuBar (doing w->hide()). In Qt5 a hidden menubar (and probably any other widget/ui object) does not get any events when being hidden. appmenu-qt5 is a simple plugin that doesn't do any of that handling. Actually I'm not even sure if it's easily possible to add shortcut handling to a QPlatformMenuItem - the QAction wrapper around platformtheme plugins. I'll look into that in detail tomorrow, maybe we might be able to resolve this somehow without patches upstream.