1593644 - Enable dom.webnotifications.requireuserinteraction on Release

Status: RESOLVED FIXED

(Core :: DOM: Push Subscriptions, task, P1)

Comment 1

[Johann Hofmann [:johannh]]

Attachment: Bug 1593644 - Enable dom.webnotifications.requireuserinteraction on Release. r=ehsan

Comment 2

[Johann Hofmann [:johannh]]

Attachment: Bug 1593644 - Also enable permissions.desktop-notification.postPrompt.enabled. r=ehsan

This is being enabled alongside dom.webnotifications.requireuserinteraction to ensure
a good user experience when automatically denying notification prompts.

Depends on D51595

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cd21f493559b
Enable dom.webnotifications.requireuserinteraction on Release. r=Ehsan
https://hg.mozilla.org/integration/autoland/rev/78ff9f5d93b7
Also enable permissions.desktop-notification.postPrompt.enabled. r=Ehsan

Comment 4

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/cd21f493559b
https://hg.mozilla.org/mozilla-central/rev/78ff9f5d93b7

Comment 5

[Kohei Yoshino]

Posted site compatibility note: https://www.fxsitecompat.dev/en-CA/docs/2019/requesting-notification-permission-now-requires-user-interaction/

Comment 6

[Julien Cristau [:jcristau]]

This is probably worth including in firefox 72 release notes?

Comment 7

[Johann Hofmann [:johannh]]

Yup

Release Note Request (optional, but appreciated)
[Why is this notable]: These prompts have been annoying users for quite a while
[Affects Firefox for Android]: Yes
[Suggested wording]: As a measure against permission prompt spam, websites can now only show notification permission prompts after the user interacted with the page.
[Links (documentation, blog post, etc)]: Blog Post, Developer Blog Post

Comment 8

[Chris Mills [:cmills]]

I think I've got the documentation done for this change; see https://github.com/mdn/sprints/issues/2464#issuecomment-564641213 for the full details.

Let me know if you'd like to see any further changes done. Thanks!