From 794424d6ad0a2b9a6eb29867473a6281a28b9b32 Mon Sep 17 00:00:00 2001 From: Till JS Date: Fri, 3 Apr 2026 11:36:26 +0200 Subject: [PATCH] fix(ui): open all AppDrawer apps in new tab All app clicks in the PillNav app drawer now open in a new tab via window.open('_blank'). Previously internal URLs used window.location.href (same tab navigation) which was confusing in the unified app context. Co-Authored-By: Claude Opus 4.6 (1M context) --- .../shared-ui/src/navigation/AppDrawer.svelte | 20 ++----------------- 1 file changed, 2 insertions(+), 18 deletions(-) diff --git a/packages/shared-ui/src/navigation/AppDrawer.svelte b/packages/shared-ui/src/navigation/AppDrawer.svelte index 944564977..21b990026 100644 --- a/packages/shared-ui/src/navigation/AppDrawer.svelte +++ b/packages/shared-ui/src/navigation/AppDrawer.svelte @@ -65,25 +65,9 @@ if (onAppClick) { onAppClick(app, event); - } else if ( - event && - (event.ctrlKey || event.metaKey) && - onOpenInPanel && - app.url && - !app.isCurrent - ) { - onOpenInPanel(app.id, app.url); - } else if (app.isCurrent) { - window.location.href = '/'; } else if (app.url) { - const isInternal = - app.url.startsWith('/') || - new URL(app.url, window.location.origin).origin === window.location.origin; - if (isInternal) { - window.location.href = app.url; - } else { - window.open(app.url, '_blank', 'noopener,noreferrer'); - } + // Always open in new tab + window.open(app.url, '_blank', 'noopener,noreferrer'); } close();