Investigate supporting `Notification.requireInteraction` for Windows native notifications
Categories
(Toolkit :: Alerts Service, task)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox117 | --- | fixed |
People
(Reporter: nrishel, Assigned: saschanaz)
References
Details
(Keywords: dev-doc-complete)
Attachments
(3 files)
The machinery necessary to support Notification.requireInteraction seems to exist but was explicitly disabled due to poor interaction with XUL notifications. In particular the issues seemed to be with persistently visible notifications blocking further notifications.
Windows native notifications have different semantics and may not experience the same issues, so we should investigate whether hooking up requireInteraction is viable.
|
||
Updated•10 months ago
|
|
Assignee | |
Comment 1•10 months ago
|
||
AFAICT we already have code in widget/windows to support requireInteraction via scenario=reminder, but that doesn't seem to work even on Nightly. 🤔
|
|
Assignee | |
Comment 2•10 months ago
|
||
Chrome has this comment: https://source.chromium.org/chromium/chromium/src/+/main:chrome/browser/notifications/win/notification_template_builder.cc;l=122-123;drc=01a77f1d907d16b49466798721936542a0cd26fe
Note: If the notification doesn't include a button, then Windows will ignore the Reminder flag. See EnsureReminderHasButton below.
Hah, that wasn't in Microsoft docs.
|
|
Assignee | |
Comment 3•10 months ago
|
||
|
|
Assignee | |
Comment 4•10 months ago
|
||
Depends on D183516
|
|
Assignee | |
Updated•10 months ago
|
|
||
Updated•10 months ago
|
|
|
||
Updated•10 months ago
|
|
|
Assignee | |
Comment 5•10 months ago
|
||
Depends on D183517
Pushed by krosylight@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/39012036cf49 Part 1: Add a default dismiss button when there's no action r=nrishel,fluent-reviewers,bolsson https://hg.mozilla.org/integration/autoland/rev/e0afefa32c0e Part 2: Enable requireInteraction by default on Windows r=smaug https://hg.mozilla.org/integration/autoland/rev/0c549a545b3f Part 3: Add a manual test r=smaug
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/41105 for changes under testing/web-platform/tests
|
||
Comment 8•10 months ago
|
||
Backed out for causing failures at test_windows_alert_service.js.
Backout link: https://hg.mozilla.org/integration/autoland/rev/b88d1706829cfda403c18c6a8a4bc1a9a4bc4555
Failure log: https://treeherder.mozilla.org/logviewer?job_id=423145352&repo=autoland&lineNumber=2256
Upstream PR was closed without merging
|
|
Assignee | |
Updated•10 months ago
|
|
|
||
Updated•10 months ago
|
|
|
Assignee | |
Updated•9 months ago
|
|
||
Comment 10•9 months ago
|
||
Pushed by krosylight@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0d49a2b0890a Part 1: Add a default dismiss button when there's no action r=nrishel,fluent-reviewers,bolsson https://hg.mozilla.org/integration/autoland/rev/74f21a0ae212 Part 2: Enable requireInteraction by default on Windows r=smaug https://hg.mozilla.org/integration/autoland/rev/38f32e096d8b Part 3: Add a manual test r=smaug
|
||
Comment 11•9 months ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/0d49a2b0890a
https://hg.mozilla.org/mozilla-central/rev/74f21a0ae212
https://hg.mozilla.org/mozilla-central/rev/38f32e096d8b
Upstream PR merged by moz-wptsync-bot
|
|
Assignee | |
Updated•9 months ago
|
|
||
Comment 13•8 months ago
|
||
MDN doc changes for this (BCD & relnote) can be tracked in the following GitHub issue: https://github.com/mdn/content/issues/28756
|
|
||
Updated•8 months ago
|
|
||
Updated•3 months ago
|
Description
•