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•1 year ago
|
Assignee | ||
Comment 1•1 year 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•1 year 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•1 year ago
|
||
Assignee | ||
Comment 4•1 year ago
|
||
Depends on D183516
Assignee | ||
Updated•1 year ago
|
Updated•1 year ago
|
Updated•1 year ago
|
Assignee | ||
Comment 5•1 year 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•1 year 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•1 year ago
|
Updated•1 year ago
|
Assignee | ||
Updated•1 year ago
|
Comment 10•1 year 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•1 year 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•1 year ago
|
Comment 13•11 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•11 months ago
|
Updated•6 months ago
|
Description
•