meven added a comment.
In D24208#548272 <https://phabricator.kde.org/D24208#548272>, @ngraham wrote: > In D24208#548153 <https://phabricator.kde.org/D24208#548153>, @broulik wrote: > > > Bluetooth could provide some better feedback inside the plasmoid. > > > Agreed, notifications shouldn't be used when the app/window/UI/whatever is still open; feedback should be provided there instead. I disagree plasmoid should avoid sending notifications. Since very often their behavior is by nature asynchronous the result of an operation can happen after the popup is closed. And having two code path for plasmoid to notify the user depending or not whether they are displayed is not recommendable. Because it would complicate the implementation and would make user experience inconsistent. > > >> Pointless confirmation notifications in plasma-nm were the main reason for this patchset. > > I think the same applies to plasma-nm. See also https://bugs.kde.org/show_bug.cgi?id=409774 This is a separate issue : the plasmoid should mute itself in this case as the notification is the results of user interaction. Hidding the notification behind a popup is at best a workaround. The issue I report here is not limited to notification originating from tray popup plasmoid. What if you receive an email while changing your audio setup ? You miss the notification until you are done with your setup, that's adding more chances to miss the notification. More generally displaying a popup should not hide notification IMO (and notifications should rarely be hidden). So yes @broulik I would be happy to patch this but I am not a specialist and it would take me quite some effort. 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 ? So I thought you would consider adding it somewhere on your infinite TODO list ^^ 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) In the end I am happy opening a notification bug, to let us deal with it it in due times. This is probably more appropriate than discussing this here. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D24208 To: broulik, #plasma, #vdg, ngraham, mart Cc: 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
