[AppMenuModel] Monitor window properties
ClosedPublic

Authored by broulik on Jan 3 2018, 11:20 AM.

Details

Summary

In case the application sets the menu after the window is shown like is the case with Firefox.

Test Plan

Started Firefox, window showed up, global menu applet showed its menu right away.
Before I had to focus another window first and go back.
A bit meh having a native event filter on qApp, though, at least it's only there if a window does not have a menu.

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Jan 3 2018, 11:20 AM
Restricted Application added a project: Plasma. · View Herald TranscriptJan 3 2018, 11:20 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Jan 3 2018, 11:20 AM
davidedmundson accepted this revision.Jan 4 2018, 6:51 PM
This revision is now accepted and ready to land.Jan 4 2018, 6:51 PM
This revision was automatically updated to reflect the committed changes.