broulik retitled this revision from "RFC: Clean up KAppMenu" to "Clean up KAppMenu". broulik updated the test plan for this revision. broulik updated this revision to Diff 7784. broulik added a comment.
- Cache atom to avoid roundtrip to the x server - Add "reconfigured" signal so interested parties (like KWin) get notified when the app menu config changes - Drop Gtk Icons mapping - it's not like GTK would support DBusMenu anymore anyway... REPOSITORY rPLASMAWORKSPACE Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3086?vs=7727&id=7784 REVISION DETAIL https://phabricator.kde.org/D3086 AFFECTED FILES appmenu/CMakeLists.txt appmenu/appmenu.cpp appmenu/appmenu.h appmenu/appmenu_dbus.cpp appmenu/appmenu_dbus.h appmenu/glowbar.cpp appmenu/glowbar.h appmenu/gtkicons.h appmenu/kdbusimporter.h appmenu/menubar.cpp appmenu/menubar.h appmenu/menubutton.cpp appmenu/menubutton.h appmenu/menuimporter.cpp appmenu/menuimporter.h appmenu/menuwidget.cpp appmenu/menuwidget.h appmenu/org.kde.kappmenu.xml appmenu/shadows.cpp appmenu/shadows.h appmenu/topmenubar.cpp appmenu/topmenubar.h appmenu/verticalmenu.h EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma, mart Cc: graesslin, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas