1257373 - Notifications fail to cause phone to chime or vibrate

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: General, defect, P5)

Description

[Edwin Wong [:edwong]]

On Fx Android
1. goto: https://people.mozilla.org/~mnoorenberghe/w3c_notifications.htm
or https://people.mozilla.org/~ewong2/push-notification-test/
2. trigger a web notification or push notification

actual:
no chime or vibrate upon receiving notification

expected:
phone should chime and vibrate. Chrome does this for push notifications (don't seem to support web notifications on android)

Comment 1

[Edwin Wong [:edwong]]

This doesn't seem to be a regression, but with the increase use of push notification rolling out in nightly, we should consider supporting this.

Comment 2

[:Margaret Leibovic]

Does the spec give websites the ability to choose a noise level, the way native app developers have the ability to do that?

This seems like it has the potential to be really obnoxious.

NI to Barbara to get this on her radar.

Comment 3

[:Margaret Leibovic]

"When updating a notification, sites can specify whether the device should alert the user with sound or vibration, or remain silent."

http://blog.chromium.org/2016/03/chrome-50-beta-push-notification.html

Comment 4

[Barbara Bermes [:barbara] - NI please!]

What do we have to do in order to make this work, as Edwin mentioned Chrome on Android doesn't support it either? Does it for iOS? Trying to understand what piece of browser piece needs to support this?

Comment 5

[Edwin Wong [:edwong]]

:barbara - I meant Chrome DOES support vibration/chime for push notifications but for general web notifications (without push from 2-3yrs ago)they've dropped support altogether

Comment 6

[Edwin Wong [:edwong]]

bryan - we should be explicit about how we want to support this in Fx48, which will be our first android release with push notifications

Comment 7

[Bryan Clark (DevTools PM) [:clarkbw]]

Just so we're clear on the state of push and (interrupting) notifications.

Chrome on Android supports this spec.
Chrome on iOS? (unclear)

I've added the user story to this bug.  We're requesting that Android support ringing or vibrating the device as specified by the push notification.  This change will keep us compatible with Chrome on Android and give developers the right level of control to make different types of push notifications relevant as needed.

We need to investigate iOS support, if Chrome doesn't yet support this on iOS it can be an advantage for us and if they already to then we'll need to add it to remain compatible across mobile operating systems.

@barbara, This is an important part of push support for mobile to remain compatible with modern mobile browsers. I'd like to get this in the backlog and see if Fx48 is possible.

Comment 8

[Andrew Overholt [:overholt]]

Web Notifications spec: https://notifications.spec.whatwg.org/

Comment 9

[:Margaret Leibovic]

(In reply to Barbara Bermes [:barbara] from comment #9)
> margaret, I assume this requires frontend UI work, is this something we
> could squeeze in after all the mobile focus updates?
> 
> If not, is another team able to work on this instead?
> 
> While I agree we should stay competitive, I wonder if this is really that
> urgent at this point, due to all the re-org?

This is a web API, so this is platform work. There is no UI.

Yes, you'll need to use Android APIs, but platform people can do that. jchen or someone on the DOM team should be able to implement this.

Comment 10

[Barbara Bermes [:barbara] - NI please!]

margaret, I assume this requires frontend UI work, is this something we could squeeze in after all the mobile focus updates?

If not, is another team able to work on this instead?

While I agree we should stay competitive, I wonder if this is really that urgent at this point, due to all the re-org?

Comment 11

[Andrew Valencia]

Consider whether 1236902 is a dup of this (or vice versa)?

Comment 12

[Andrew Valencia]

> I wonder if this is really that urgent at this point, due to all the re-org?

Without this, FF on Android is a non-starter for *any* real time messaging or apps w. state updates.
Chrome only supports the Notification API on Service Workers, which I've just added to my
own messaging so I can get full support on Android:

    https://github.com/vandys/-webXMPP

People are not going to be OK with turning on their phone every other minute.  Please, please
fix this.  Thanks.

Comment 13

[Jovan Gerodetti]

I really need this. Right now notifications are almost useless.

Comment 14

[Jovan Gerodetti]

This could be fixed quite simple in:
https://hg.mozilla.org/mozilla-central/file/tip/mobile/android/base/java/org/mozilla/gecko/notifications/NotificationClient.java#l149

Add the following [1]:

> .setDefaults(DEFAULT_ALL)

[1] https://developer.android.google.cn/reference/android/support/v4/app/NotificationCompat.Builder.html#setDefaults(int)


Additional notification options also could be supported easily, like re-notify, custom vibration patterns, etc.

Comment 15

[Jovan Gerodetti]

https://developer.mozilla.org/en-US/docs/Web/API/Notification/Notification#Parameters

Comment 16

[Barbara Bermes [:barbara] - NI please!]

Joe, this might be something we can put on the Trello board for Fennec....

Comment 17

[Joe Cheng [:jcheng] (please needinfo)]

Added https://trello.com/c/IheJQw3v/188-notifications-fail-to-cause-phone-to-chime-or-vibrate

Comment 18

[Jovan Gerodetti]

In an other bug the default sound for notifications has been enabled, but vibration is still missing. It would be great if you could add this too.

Comment 19

[Andrew Valencia]

Enabling sound was great, please set the bit for blinking the notification LED as well.
Otherwise if you miss the sound you won't know about messages waiting until you turn
on the screen.

Comment 20

[Jovan Gerodetti]

IS this going to get fixed?

Comment 21

[Andrew Valencia]

My "daily driver" is Android with Fennec, using long polling and HTML5 Notifications for email/SMS/XMPP/telemtry.
Your adding of sound for notifications was great; I can provide a user story if that can help you think about
vibration and LED as well.

Comment 22

[Barbara Bermes [:barbara] - NI please!]

Currently not scheduled, will updated once priorities change.

Comment 23

[Firefox Bug Husbandry Bot]

Re-triaging per https://bugzilla.mozilla.org/show_bug.cgi?id=1473195

Needinfo :susheel if you think this bug should be re-triaged.

Comment 24

[Sylvestre Ledru [:Sylvestre]]

Don't think we will do it, closing.

Comment 25

[Andrew Valencia]

ACtually, this works now, so if it helps your stats, you might want to close it as a dup instead.

Oh, BTW, it works really well. With full user control of light/vibe/sound. Nice!