search

mozilla / chat.mozillafoundation.org

Open source Slack-alternative in Golang and React - Mattermost
http://mattermost.org
Other
5 stars 6 forks source link

Allow user to change push notification sounds #54

Open edrushka opened 8 years ago

edrushka commented 8 years ago

Not sure if this is possible, but @alanmoo and I were looking at how to do this (in FF) and it seems complicated.

Pomax commented 8 years ago

Going over https://developer.mozilla.org/en/docs/Web/API/notification, it looks like we should (without any idea how much work it is =D) be able to control the sound resource used, as well as whether the notification should have sound or is silent.

edrushka commented 8 years ago

Blessed silence would be a nice option ;)

alanmoo commented 8 years ago

You can turn them off in the webapp, you just can't change them.

edrushka commented 8 years ago

@alanmoo can I turn off the sound but still get a little popup notification? I don't see that option in Mattermoz and in FF my only options are 'allow' or 'don't allow' notifications. I can't customize from the FF end.

alanmoo commented 8 years ago

@edrushka Do you see both these options in notifications? Set them as I've got and it should do what you want. screenshot 2016-02-18 15 40 24

edrushka commented 8 years ago

Interesting. I don't have the option to turn sounds off. I'm using FF 44.0.2

screenshot 2016-02-18 15 47 30

alanmoo commented 8 years ago

Ah, that does appear to be disabled in FF, even though the messaging in electron-mattermost says "Desktop notification sounds are available on Firefox, Safari, Chrome, Internet Explorer, and Edge." In any case, this is an issue related to the browser, not the application.

Pomax commented 8 years ago

looks like FF supports this just fine, but the web interface isn't letting the browser know it needs that authorised? http://www.bennish.net/web-notifications.html shows it working fine in FF for me, but then in mattermoz it does naught.

alanmoo commented 8 years ago

Interesting. Maybe something has changed in FF since this feature was added.

-Alan

On Feb 18, 2016, 6:26 PM -0800, Mike Kamermansnotifications@github.com, wrote:

looks like FF supports this just fine, but the web interface isn't letting the browser know it needs that authorised?http://www.bennish.net/web-notifications.htmlshows it working fine in FF for me, but then in mattermoz it does naught =x

— Reply to this email directly orview it on GitHub(https://github.com/mozilla/mattermoz/issues/54#issuecomment-186019336).