<table><tr><td style="">broulik updated the summary for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-nlnxcvbcysgp5wq/" rel="noreferrer">(Show Details)</a><br />broulik updated this revision to Diff 7727.<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">Remove GtkIcons, it's not like GTK applications support this stuff anymore...</li>
<li class="remarkup-list-item">Introduce showMenu signal so KWin can reliably know whether the menu actually showed (sometimes when the menu doesn't show up for reasons, KWin would get stuck thinking it's open, so we explicitly emit a signal when we popup'd the menu)</li>
<li class="remarkup-list-item">Listen to ItemActivationRequested signal we emit in Plasma-Integration, when called with 0,0 arguments we emit a generic "show menu" request</li>
<li class="remarkup-list-item">Re-introduce Unity about to show hack but it doesn't actually fix the situation with Firefox for me :/</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>Massively cleanup KAppMenu by removing outdated and unused code.<span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);"> The global menu has been removed as it will eventually be provided by a plasmoid (famous last words). Also many of the workarounds (like the unity about to show hack) have been removed,</span> <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">unfortunately the issue with Firefox seems to</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">The global menu has been removed as it will eventually</span> be <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">still the case</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">provided by a plasmoid (famous last words)</span>.<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=7572&id=7727" rel="noreferrer">https://phabricator.kde.org/D3086?vs=7572&id=7727</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>plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>