Skip to content

fix(web-notifications): open notification link in a new tab - #3537

Merged
nickvergessen merged 2 commits into
masterfrom
fix/noid/web-notification-new-tab
Oct 28, 2026
Merged

nickvergessen merged 2 commits into
masterfrom
fix/noid/web-notification-new-tab

Conversation

@Antreesy

Copy link
Copy Markdown
Collaborator

Follow-up to #3478 (comment)

  • Since PR, with closed tab, webpush notifications coming from service worker and handled by this logic (open a new tab)
  • If any NC tab is opened, it generates a browser notification, click on it replaced content the current page that is considered main process
    • it can drop unsaved work or an ongoing call in that tab.
  • Now it always open a new tab

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (ClaudeCode:claude-opus-5-5)

A click on a browser notification replaced the current page, which
dropped unsaved work or an ongoing call in that tab.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy
Antreesy force-pushed the fix/noid/web-notification-new-tab branch from ac4bd9d to a03f0ec Compare October 28, 2026 12:45
@Antreesy

Copy link
Copy Markdown
Collaborator Author

/backport a03f0ec to stable35

@nickvergessen

Copy link
Copy Markdown
Member

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nickvergessen
nickvergessen merged commit e4bf156 into master Oct 28, 2026
34 checks passed
@nickvergessen
nickvergessen deleted the fix/noid/web-notification-new-tab branch October 28, 2026 12:55
@nickvergessen

Copy link
Copy Markdown
Member

/backport a03f0ec to stable34

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants