Skip to content
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

Positive on Declarative Web Push #1190

Merged
merged 3 commits into from
Mar 20, 2025
Merged

Positive on Declarative Web Push #1190

merged 3 commits into from
Mar 20, 2025

Conversation

saschanaz
Copy link
Contributor

Closes #1176

activities.yml Outdated
id: declarative-web-push
issue: 1176
rationale: |
Mozilla is happy with the overall idea of Declarative Web Push. We believe having a declarative way to subscribe to push notifications will help users by reducing power usage on mobile devices and also help web authors by reducing required code size. We also believe that this introduces a way for browsers to suppress potentially unwanted push notifications from tracking users by conditionally removing its ability to use local state for mutable notifications.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So... I don't believe Apple's claims about power usage here. It's obviously less power consumption to not run code, but I don't find it convincing that instantiating and running a small SW is significant relative to activating the device radio, the speaker/haptics, or other costs that come with notifications.

What I do think is that this model of notifications is probably better overall than what we originally specified. Having a clear path to a notification, with the option to intercept and update application state, is useful. Having the option be truly optional in the sense that the browser might choose not to invoke it, is also good.

I do think that this is an incomplete solution, in that having no declarative way to dismiss a notification using this mechanism is problematic. Mostly from the perspective of not enabling the creation of annoyances.

As for the rest of the text in this chunk, I would suggest that you post the other stuff on the issue, rather than here. It's enough to just say that we think this is good.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually do we need a PR at all or is adding a tag to the issue enough?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commenting in the issue and adding a label is enough. PR is needed if we want to include a description/rationale in the dashboard.

@zcorpan zcorpan merged commit 0c61259 into mozilla:main Mar 20, 2025
1 check passed
@saschanaz saschanaz deleted the dwp branch March 20, 2025 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Push API] Declarative Web Push
3 participants