mirror of
https://github.com/binwiederhier/ntfy.git
synced 2026-05-09 16:35:53 +02:00
[PR #899] [MERGED] fix(pwa): hide install prompt on macOS 14 safari #1453
Labels
No labels
ai-generated
android-app
android-app
android-app
🪲 bug
build
build
dependencies
docs
enhancement
enhancement
🔥 HOT
in-progress 🏃
ios
prio:low
prio:low
pull-request
question
🔒 security
server
server
unified-push
web-app
website
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ntfy#1453
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/binwiederhier/ntfy/pull/899
Author: @nihalgonsalves
Created: 9/27/2023
Status: ✅ Merged
Merged: 10/2/2023
Merged by: @binwiederhier
Base:
main← Head:ng/fix-safari-17-sonoma📝 Commits (4)
66ef28cfix(pwa): hide install prompt on macos 14 safari5913142fix: remove deprecated nodesource scriptd33ededdocs: remove Safari sound warning65dff6eMerge branch 'main' into ng/fix-safari-17-sonoma📊 Changes
4 files changed (+28 additions, -9 deletions)
View changed files
📝
Dockerfile-build(+11 -7)➕
docs/static/img/pwa-install-macos-safari-add-to-dock.png(+0 -0)📝
docs/subscribe/pwa.md(+7 -0)📝
web/src/app/Notifier.js(+10 -2)📄 Description
macOS 14 has support for "Add to Dock", which added the
window.navigator.standaloneproperty to macOS. This was previously only on iOS and we relied on it to display the "install required" prompt.Since the limitation of needing an installed web app does not exist on desktop Safari, ntfy therefore wrongly stops you from using the web app notifications. This changes the heuristic to only display the install prompt when the push manager API is not available.
I also added a screenshot of the new Safari "Add to Dock" action to the docs.
While testing it I saw a deprecation warning when using the node source script, so I fixed that too - I can move that to another PR if wanted.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.