<table><tr><td style="">broulik updated the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-k4yfgsd5qgv7fno/" rel="noreferrer">(Show Details)</a><br />broulik updated this revision to Diff 7572.<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><p>deleteLater() the importer - fixes activating menu items</p></div></div><br /><div><strong>EDIT DETAILS</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>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.<br />
<br />
<span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">The about to show hack seems to still be neccessary for Firefox - as soon as opening a sub menu it</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">The about to show hack seems to still be neccessary for Firefox and Chrome - as soon as opening a sub menu it closes in both of them.</span> <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">closes in Firefox.</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">Needs further investigation,</span> <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">Needs further investigation.<br />
<br />
I also just noticed that triggering actions in the menu doesn't actually work right now after the migration away from window IDs</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">probably reintroducing the aforementioned hack.</span> <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">:)</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">:/</span></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=7570&id=7572" rel="noreferrer">https://phabricator.kde.org/D3086?vs=7570&id=7572</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/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<br /><strong>Cc: </strong>plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>