<table><tr><td style="">davidedmundson requested changes to this revision.<br />davidedmundson added a comment.<br />This revision now requires changes to proceed.
</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/D8519" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>The removal of the force activation is done since according to kwin maintainer it is wrong and must be done by the SysTray itself.</p></blockquote>

<p>But according the the status notifier item maintainer (defacto me), we cannot do this. It will break too many things.</p>

<p>The SNI spec is used by Unity 7, Gnome, and Plasma. Can we guarauntee their trays do this? (in fact even Plasma doesn't currently do this!)</p>

<p>I know it's putting you in the middle of an argument, which is unfair on you, sorry.</p>

<p>forceActiveWindow may be bad, but it works. It's not really focus stealing if we explicitly know the user performed the action.  In an ideal world the original SNI spec would have passed a timestamp...and blah blah, but yeah.</p>

<p>For Wayland this won't work and will do nothing, which is fine. We do need to fix that properly in a different manner, but lets not break X in the meantime.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R289 KNotifications</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8519" rel="noreferrer">https://phabricator.kde.org/D8519</a></div></div><br /><div><strong>To: </strong>mkoller, davidedmundson, graesslin<br /><strong>Cc: </strong>Frameworks<br /></div>