<table><tr><td style="">meven 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/D24208">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D24208#548272" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: line-through;">D24208#548272</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D24208#548153" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: line-through;">D24208#548153</a>, <a href="https://phabricator.kde.org/p/broulik/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@broulik</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>Bluetooth could provide some better feedback inside the plasmoid.</p></div>
</blockquote>
<p>Agreed, notifications shouldn't be used when the app/window/UI/whatever is still open; feedback should be provided there instead.</p></div>
</blockquote>
<p>I disagree plasmoid should avoid sending notifications.<br />
Since very often their behavior is by nature asynchronous the result of an operation can happen after the popup is closed.<br />
And having two code path for plasmoid to notify the user depending or not whether they are displayed is not recommendable.<br />
Because it would complicate the implementation and would make user experience inconsistent.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;">
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Pointless confirmation notifications in plasma-nm were the main reason for this patchset.</p></blockquote>
<p>I think the same applies to plasma-nm. See also <a href="https://bugs.kde.org/show_bug.cgi?id=409774" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=409774</a></p></blockquote>
<p>This is a separate issue : the plasmoid should mute itself in this case as the notification is the results of user interaction.<br />
Hidding the notification behind a popup is at best a workaround.</p>
<p>The issue I report here is not limited to notification originating from tray popup plasmoid.<br />
What if you receive an email while changing your audio setup ?<br />
You miss the notification until you are done with your setup, that's adding more chances to miss the notification.<br />
More generally displaying a popup should not hide notification IMO (and notifications should rarely be hidden).</p>
<p>So yes <a href="https://phabricator.kde.org/p/broulik/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@broulik</a> I would be happy to patch this but I am not a specialist and it would take me quite some effort.<br />
On the other end the feature has been recently added and should be refined IMO (I am trying to make this point): how long before we get bug reports ?<br />
So I thought you would consider adding it somewhere on your infinite TODO list ^^</p>
<p>I am sorry I react only now but I saw this issue with this arising when the review was opened but did not react due to LIFE(tm)</p>
<p>In the end I am happy opening a notification bug, to let us deal with it it in due times.<br />
This is probably more appropriate than discussing this here.</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/D24208">https://phabricator.kde.org/D24208</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, VDG, ngraham, mart<br /><strong>Cc: </strong>meven, mart, ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra<br /></div>