lxqt-notificationd icon indicating copy to clipboard operation
lxqt-notificationd copied to clipboard

[Bug] Reading a default duration of 0 from the config file will incorrectly display as 10 sec upon restarting

Open snailfart opened this issue 4 years ago • 3 comments

If the previous value is 0, selecting "Advanced Settings" in lxqt-config-notificationd always displays the predefined duration of 10 sec. This issue is however only cosmetical; it doesn't reset the timeout I set previously until alter it manually.

lxqt-notificationd Version: 0.15.0

snailfart avatar Sep 20 '20 16:09 snailfart

I agree that its not a good approach. I believe the idea was to prevent 0 sec. but in that case it should save the 10 sec. timeout.

The thing is, if you don't want to see notifications from a specific app you should blacklist it and if you don't want to show any notification then a notification-daemon should not be running but thats not a good option (some apps will not work if there isnt a notification-daemon running, i.e. electron apps) perhaps implementing a "silence mode" would be a good idea?

In that case the daemon would receive the notification but not show it, the same as setting timeout to 0.

zhrexl avatar Sep 23 '20 23:09 zhrexl

I understand but this was not my intention at all, I wanted the notifications to never fade until I click them away, which worked very well, I can even set it to 0 through the GUI, it will just return to 10 sec after restarting the application.

snailfart avatar Sep 24 '20 22:09 snailfart

Oh, I see. I misunderstood what you wanted to achieve. I'll take a look to see what can be improved.

zhrexl avatar Sep 25 '20 03:09 zhrexl