Closed
Bug 1378445
Opened 7 years ago
Closed 7 years ago
notifications on firefox for android are silent
Categories
(Firefox for Android Graveyard :: General, enhancement)
Firefox for Android Graveyard
General
Tracking
(firefox56 fixed)
RESOLVED
FIXED
Firefox 56
Tracking | Status | |
---|---|---|
firefox56 | --- | fixed |
People
(Reporter: bkelly, Assigned: cnevinchen)
Details
Attachments
(2 files)
I use the telegram messaging site on my android device quite a lot. The site uses push messages and notifications. One of my biggest annoyances, though, is that firefox for android does not produce any sound/vibration when we display notifications on android. This greatly reduces the utility of this site on firefox compared to installing the native app.
Can we please enable sound/vibration on our notifications? If a user does not like them they can disable in the settings app like they do for all other applications.
(Of course, they won't be able to disable sound/vibration for only one site. Just for all of firefox.)
James, I think we spoke about this at the all hands last week. Just an NI since it sounded like you agreed this was a bug.
Flags: needinfo?(snorp)
Nevin, can you take a look? We should just allow the normal notification sound to play. Something going wrong in NotificationHelper.
Flags: needinfo?(snorp) → needinfo?(cnevinchen)
Assignee | ||
Comment 2•7 years ago
|
||
I just came back from PTO and will take a look at this.
Assignee: nobody → cnevinchen
Flags: needinfo?(cnevinchen)
Assignee | ||
Comment 3•7 years ago
|
||
I just tested it using Nightly on Nexus 5 Android 6.0.1
I can hear the sound when I got a message.
Could you please let me know more about your device?
I'll also test it further tomorrow when I get in the office.
Flags: needinfo?(bkelly)
Reporter | ||
Comment 4•7 years ago
|
||
I'm on a nexus 5x running Android 7.1.2. See the attached screenshot showing that my notifications settings are not set to silent for nightly. I get notification sounds for other apps.
Flags: needinfo?(bkelly)
Comment hidden (mozreview-request) |
Comment 6•7 years ago
|
||
mozreview-review |
Comment on attachment 8887549 [details]
Bug 1378445 - Add sound for notifications.
https://reviewboard.mozilla.org/r/158414/#review163830
Attachment #8887549 -
Flags: review?(snorp) → review+
Pushed by nechen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/04a5e514114e
Add sound for notifications. r=snorp
Comment 8•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Comment 9•7 years ago
|
||
What about vibration? It would be great if Firefox also uses vibration for it's notifications.
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•