Respect OS Do Not Disturb mode

[element-hq]

Respect OS Do Not Disturb mode

[otheus]

Just to be clear, the preferred way on MacOS is to use the notifications API, so that do-not-disturb mode and other OS behavior is consistent.

[pepyakin]

Op here. FWIW, for me this was solved by migrating to the web version, to PWA app within Chrome to be exact.

[JMoVS]

Hi @HarHarLinks , the mentioned specs are unrelated to this bug. They are a great feature, but the bug here is about being a good platform citizen as a desktop app.

[Conan-Kudo]

I would really like this feature to be implemented. Element is the only application I run on my desktop (Fedora Linux 38 with KDE Plasma Desktop) that does not respect the system-wide do-not-disturb setting. Even Slack (also an Electron app) respects the setting properly.

[otheus]

@tawfiek In general, I think this is a good idea. It always seems suspect to me to rely on 3rd party libraries whose code has not been audited and which cannot be absolutely authenticated. AFter all, this is communication software for private data...

[Half-Shot]

The Do Not Disturb mode on gnome is the dconf setting /org/gnome/desktop/notifications/show-banners. Element notifs should already be blocked by the setting, but sounds won't be. We will need to find a library that allows us to listen on dbus for when this value is updated.

[pszypowicz]

There seem to be a user space lib to detect dnd mode on mac: https://github.com/felixrieseberg/macos-notification-state

My workaround is to close element app when I'm expecting to focus on work. Not ideal one may say /sarcasm

[aaronraimist]

:/ I don't think Electron has a built in way to tell if DND is enabled. Looks like Electron recommends using this library https://github.com/felixrieseberg/electron-notification-state which only has one commit... maybe it's just really stable and good at it's current state?

[lampholder]

That felixrieseberg library just wraps two other libraries for fetching mac/windows notification states (also written by felixrieseberg, and with 7 and 9 commits respectively). Scanning the code it does look feasible that it might just work?

[yajo]

As explained in element-hq/element-web#13941, the same happens in linux.

[Half-Shot]

#133 should hopefully address folks on GNOME

[t3chguy]

Electron does not support accessing that API seemingly, but closest we can get is to try and read the DND state using https://www.electronjs.org/docs/tutorial/notifications#do-not-disturb--session-state

[tawfiek]

Electron does not support accessing that API seemingly, but closest we can get is to try and read the DND state using https://www.electronjs.org/docs/tutorial/notifications#do-not-disturb--session-state

Electron refers to the same package of felixrieseberg in there official docs, but in the package's GitHub repo I found more than one issue complaint that this package is deprecated for the newer versions of node, I think it's a bad Idea to rely on it in element.
maybe we can fork it update it's dependencies and republish it, then use it into our codebase or ...
"it" refers to the packages that this lib wrap, since it just a warper for : macos-notification-state and windows-notification-state written with the same guy as @lampholder said

[germain-gg]

We have a do not disturb mode in user settings now. We could hook the OS level setting to honour that user preference.

From what I understand we will only be able to do that on the Electron side as the media queries are not exposing any of that

[HarHarLinks]

We have a do not disturb mode in user settings now.

This was removed from labs in matrix-org/matrix-react-sdk@4d7b7c4 and I miss it dearly.

[JMoVS]

surely there is a way 4 years later to use the native notification system and get do not disturb following for free, right? Is this not annoying others?

[HarHarLinks]

There is some related spec work (currently not progressing):

matrix-org/matrix-spec-proposals#3026
matrix-org/matrix-spec-proposals#3767

[uzantome]

Here's a PR from Mattermost, another electron-based messenger, implementing this:

mattermost/desktop#1629

[t3chguy]

@uzantome looks like it only supports Linux & Windows, and would create extra cost on the latter as it'd require signing extra binaries