-
Notifications
You must be signed in to change notification settings - Fork 49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Should a non-persistent notification be auto-closed or not? #204
Comments
Also nobody clears non-persistent notifications from workers 👀 (Except Firefox has no Edit: Maybe we should get something corresponding to declarative web push so that script-triggered persistent notifications won't depend on service workers either. |
Chrome and Safari (haven't tested Firefox) violate the next line as well:
Chrome on Windows and ChromeOS (haven't tested other platforms) add non-persistent notifications to the platform's "notification center". Safari does as well, though as mentioned above it removes them when the tab is navigated or closed. Chrome does not. |
From all the things this is the weirdest behavior listed to me, but we could allow this as one of the alternatives. It could use the same vague-ish definition we plan to use for I'd also be happy to remove or weaken most of the other requirements in this section. I'd probably keep the non-persistent/persistent distinction as-is as it's kinda a pain to rename and it still seems accurate lifetime-wise, at least from the perspective of the web developer. |
What is the issue with the Notifications API Standard?
Currently the spec says: https://notifications.spec.whatwg.org/#lifetime-and-ui-integrations
The actual browser behavior is different:
We had internal argument against the second behavior; non-persistent notification may also include some crucial content that user would like to keep in the notification center. My immediate impression is that such notification should use persistent way instead, but given the browser differences, I think should discuss about the behavior here for better interop.
The text was updated successfully, but these errors were encountered: