broulik created this revision.
broulik added reviewers: Plasma, davidedmundson.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  We call `updateMenu()` to cache the first level of menus in advance in global 
menu applet.
  Just move the "opened" call there to ensure it's always called both when the 
menu shows or when someone manually triggers it.

TEST PLAN
  I have working menus in Firefox now \o/
  Other apps and tray icons seem unaffected
  
  Follow up to https://phabricator.kde.org/D9631 which wasnt sufficient
  
  I checked Qt code, it doesn't do anything with "opened" signal

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D9782

AFFECTED FILES
  libdbusmenuqt/dbusmenuimporter.cpp

To: broulik, #plasma, davidedmundson
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart

Reply via email to