<table><tr><td style="">broulik retitled this revision from "RFC: Clean up KAppMenu" to "Clean up KAppMenu".<br />broulik updated the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-qehk6zuv72t7lf3/" rel="noreferrer">(Show Details)</a><br />broulik updated this revision to Diff 7784.<br />broulik added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D3086" rel="noreferrer">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Cache atom to avoid roundtrip to the x server</li>
<li class="remarkup-list-item">Add "reconfigured" signal so interested parties (like KWin) get notified when the app menu config changes</li>
<li class="remarkup-list-item">Drop Gtk Icons mapping - it's not like GTK would support DBusMenu anymore anyway...</li>
</ul></div></div><br /><div><strong>EDIT DETAILS</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div> Style=<span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">ButtonVertical</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">Decoration</span><br />
<br />
Applications like Kwrite, Dolphin, as well as VLC start exporting their manus on DBus now. The "showMenu" method on the org.kde.kappmenu dbus interface can be used to show the application menu. The menu button also shows up for Chrome and Firefox.<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMAWORKSPACE Plasma Workspace</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D3086?vs=7727&id=7784" rel="noreferrer">https://phabricator.kde.org/D3086?vs=7727&id=7784</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3086" rel="noreferrer">https://phabricator.kde.org/D3086</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>appmenu/CMakeLists.txt<br />
appmenu/appmenu.cpp<br />
appmenu/appmenu.h<br />
appmenu/appmenu_dbus.cpp<br />
appmenu/appmenu_dbus.h<br />
appmenu/glowbar.cpp<br />
appmenu/glowbar.h<br />
appmenu/gtkicons.h<br />
appmenu/kdbusimporter.h<br />
appmenu/menubar.cpp<br />
appmenu/menubar.h<br />
appmenu/menubutton.cpp<br />
appmenu/menubutton.h<br />
appmenu/menuimporter.cpp<br />
appmenu/menuimporter.h<br />
appmenu/menuwidget.cpp<br />
appmenu/menuwidget.h<br />
appmenu/org.kde.kappmenu.xml<br />
appmenu/shadows.cpp<br />
appmenu/shadows.h<br />
appmenu/topmenubar.cpp<br />
appmenu/topmenubar.h<br />
appmenu/verticalmenu.h</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, mart<br /><strong>Cc: </strong>graesslin, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>