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•2 years ago
|
| Assignee | ||
Comment 1•2 years 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•2 years 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•2 years ago
|
||
| Assignee | ||
Comment 4•2 years ago
|
||
Depends on D183516
| Assignee | ||
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
| Assignee | ||
Comment 5•2 years ago
|
||
Depends on D183517
Comment 8•2 years 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
| Assignee | ||
Updated•2 years ago
|
Updated•2 years ago
|
| Assignee | ||
Updated•2 years ago
|
Comment 10•2 years ago
|
||
Comment 11•2 years 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
| Assignee | ||
Updated•2 years ago
|
Comment 13•2 years ago
|
||
MDN doc changes for this (BCD & relnote) can be tracked in the following GitHub issue: https://github.com/mdn/content/issues/28756
Updated•2 years ago
|
Updated•2 years ago
|
Description
•