<table><tr><td style="">subdiff created this revision.<br />subdiff added reviewers: Plasma, davidedmundson.<br />subdiff set the repository for this revision to R120 Plasma Workspace.<br />subdiff added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.
</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/D4301" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This patch primarily is aimed at applications using libappindicator. For example:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Steam</li>
<li class="remarkup-list-item">Discord</li>
<li class="remarkup-list-item">Deluge</li>
</ul>

<p>libappindicator doesn't provide functionality for raising an application, but only a context menu. Since it also doesn't even provide the corresponding DBus method, we use the resulting error to try to display the context menu instead, which matches the behaviour on Unity and Gnome.</p>

<p><a href="https://www.gamingonlinux.com/forum/topic/2519" class="remarkup-link" target="_blank" rel="noreferrer">https://www.gamingonlinux.com/forum/topic/2519</a><br />
<a href="https://bugs.kde.org/show_bug.cgi?id=375351" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=375351</a></p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Manually with libappindicator and Qt based tray icons.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4301" rel="noreferrer">https://phabricator.kde.org/D4301</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/systemtray/package/contents/ui/items/StatusNotifierItem.qml<br />
dataengines/statusnotifieritem/statusnotifieritemjob.cpp<br />
dataengines/statusnotifieritem/statusnotifieritemsource.cpp<br />
dataengines/statusnotifieritem/statusnotifieritemsource.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>subdiff, Plasma, davidedmundson<br /><strong>Cc: </strong>plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>