Pings/notifications can fail if the user doesn't interact with the document first

[turt2live]

Uncaught (in promise) DOMException: play() failed because the user didn't interact with the document first. https://goo.gl/xX8pDD

This lead to me not having badges for unread messages or pings in the room list. At the very least, we should try our best to display that there was a notification even if we can't play the sound.

[turt2live]

gah, the commit caught it.

[MadLittleMods]

@turt2live I'm sure this is still possible but is this issue that valuable?

At the very least, we should try our best to display that there was a notification even if we can't play the sound.

I agree with this type of thing though but feels like a heuristic we should be thinking about when writing a lot of things.

[turt2live]

The bug here is effectively the .play() call and desktop notification call are inverted: we should be showing the desktop notification first, then we can let play() fail for the noise.

[MadLittleMods]

The bug here is effectively the .play() call and desktop notification call are inverted: we should be showing the desktop notification first, then we can let play() fail for the noise.

We already do this: src/Notifier.ts#L491-L497