lxqt / lxqt-notificationd

The LXQt notification daemon
https://lxqt.github.io
GNU Lesser General Public License v2.1
62 stars 38 forks source link

Show LXQt config's test notifications in do-not-disturb mode #294

Closed tsujan closed 2 years ago

stefonarch commented 2 years ago

BTW: it looks like save=0 and "do not disturb" together means "unlimited", if true it needs explanations or else.

tsujan commented 2 years ago

BTW: it looks like save=0 and "do not disturb" together means "unlimited"

You mean unattendedMaxNum=0 (in the config file).

Logically, the do-not-disturb mode should save notifications regardless of unattendedMaxNum. So, if doNotDisturb=true, at least up to 50 notifications will be saved, even when unattendedMaxNum=0.

The settings of unattended notifications should always be present because the user may disable the do-not-disturb mode.

Adding all this (and more) as a tooltip would have the inverse effect: it would confuse the user. The important thing is that his notifications aren't lost in the do-not-disturb mode.