STEPS TO REPRODUCE
1. activate "Do not disturb" mode in notifications applet
2. press shift+printscreen to take a screenshot with spectacle (plasma shows a notification)
3. open system settings > Notifications
4. click "Configure..." button on botton
5. select "Spectacle" in the applications list
6. uncheck "Show in do not disturb mode" and click "Apply" button
7. take a screenshot with spectacle again by pressing shift+printscreen
plasma shows notification in "Do not disturb" mode
even when "Show in do not disturb mode" related to the
source app is unchecked in notifications kcm.
do not show notifications while "Do not disturb" mode is activated
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.15.80
KDE Frameworks Version: 5.58.0
Qt Version: 5.12.0
I think this intended behavior. I see two reasons for that. First the user should know that a screenshot has been taken for security and privacy concerns depending on the screenshots because it is saved as a file. For example it could have been triggered by accidentally activating the shortcut. Second if you take a screenshot there is a high chance that you want to interact with it - for example drag and drop it from the notification.
The reasons I gave are my ideas. But I think this is working as intended because I remembering Kai saying that Spectacle should be exempt from Do not disturb while he was implementing the new notifications
Here is spectacle excluded from do not disturb
Correct, this is behaving as intended. Even in "Do Not Disturb" mode, we show so-called "critical" notifications (e.g. "your laptop is about to run out of power"). Spectacle's notifications are marked as critical for the reasons that David provided.
Well, the bug explicitly states that the setting is broken, not the fact that we have spectacle by default.
I can confirm that unchecking the setting in Spectacle doesn't work. Probably a bug in the code that writes the config, using revertToDefault.
Git commit a24166f72fbd2a3f66d49e26e7b7eb755c9fd99a by Kai Uwe Broulik.
Committed on 29/05/2019 at 11:47.
Pushed by broulik into branch 'Plasma/5.16'.
[Notifications] Check proper group for show popups in dnd mode setting
It checked the wrong one resulting in the setting not being written.
Differential Revision: https://phabricator.kde.org/D21475
M +1 -1 libnotificationmanager/settings.cpp