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.

Sign up for GitHub

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

Jump to bottom

Safari for iOS/Chrome for Android completely disallows `new Notification()` #210

Open

saschanaz opened this issue May 24, 2024 · 0 comments

Member

saschanaz commented May 24, 2024 •

edited

Loading

What is the issue with the Notifications API Standard?

Blink:
1. The constructor throws if called without pref: https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/modules/notifications/notification.cc;l=76-81;drc=494419358caf690316f160a1f27d9e771a14c033
2. Pref is disabled on Android: https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/platform/runtime_enabled_features.json5;l=2687-2692;drc=a68be90fb705c6a538a9ef550fe7bca9595e8f29
Safari:
1. Notification interface itself is behind a pref: https://searchfox.org/wubkat/rev/04bd634bf20976f70014303e66875f22f8d65946/Source/WebCore/Modules/notifications/Notification.idl#34-40
2. And it's disabled on iOS https://searchfox.org/wubkat/rev/04bd634bf20976f70014303e66875f22f8d65946/Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml#4676-4692

I understand the decision as tabs can be unloaded on mobile more frequently, but the spec does not cover this and it should.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment