D10461: GMenu-DBusMenu-Proxy

Kai Uwe Broulik noreply at phabricator.kde.org
Wed Feb 14 11:41:07 UTC 2018


broulik updated this revision to Diff 27142.
broulik edited the test plan for this revision.
broulik added a comment.


  - Support unity actions (untested)
  - Don't mandate actions to be present, it's perfectly valid for a window to have a menu only consisting of application actions or vice-versa
  - Improve action update logic, signal `ItemsPropertiesUpdated` when an action changes instead of rebuilding the layout
  - Improve menu update logic, just update the existing items when the same number of items is removed and added

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D10461?vs=27077&id=27142

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

AFFECTED FILES
  CMakeLists.txt
  gmenu-dbusmenu-proxy/CMakeLists.txt
  gmenu-dbusmenu-proxy/gdbusmenutypes_p.cpp
  gmenu-dbusmenu-proxy/gdbusmenutypes_p.h
  gmenu-dbusmenu-proxy/gmenudbusmenuproxy.desktop
  gmenu-dbusmenu-proxy/main.cpp
  gmenu-dbusmenu-proxy/menu.cpp
  gmenu-dbusmenu-proxy/menu.h
  gmenu-dbusmenu-proxy/menuproxy.cpp
  gmenu-dbusmenu-proxy/menuproxy.h

To: broulik, #plasma
Cc: rk, rilian, mtallur, ngraham, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180214/545c9dee/attachment.html>


More information about the Plasma-devel mailing list