From 7ec7eb86bf3532e3e5f45bb2e4ae8384fa67f798 Mon Sep 17 00:00:00 2001
From: Peter Ringelmann <peter.ringelmann@nextcloud.com>
Date: Wed, 6 May 2026 11:15:50 +0200
Subject: [PATCH] fix: remove new tab links -e Signed-off-by: Peter Ringelmann
 <peter.ringelmann@nextcloud.com>

---
 core/src/components/AppItem.vue           | 5 -----
 core/src/components/AppMenu.vue           | 5 -----
 core/src/tests/components/AppItem.spec.ts | 8 --------
 3 files changed, 18 deletions(-)

diff --git a/core/src/components/AppItem.vue b/core/src/components/AppItem.vue
index 1d05814feb9..1aa146febe0 100644
--- a/core/src/components/AppItem.vue
+++ b/core/src/components/AppItem.vue
@@ -11,8 +11,6 @@
 			'app-item--outlined': outlined,
 		}"
 		:href="app.href"
-		:target="newTab ? '_blank' : undefined"
-		:rel="newTab ? 'noopener noreferrer' : undefined"
 		:aria-current="app.active ? 'page' : undefined"
 		:tabindex="tabindex"
 		:title="app.name"
@@ -43,8 +41,6 @@ import { computed } from 'vue'
 
 const props = withDefaults(defineProps<{
 	app: INavigationEntry
-	/** When true, the link opens in a new tab. Launcher uses true; Dashboard reuse will use false. */
-	newTab?: boolean
 	/** When true, render the circle as an outline only (used for "More apps" / utility entries). */
 	outlined?: boolean
 	/**
@@ -54,7 +50,6 @@ const props = withDefaults(defineProps<{
 	 */
 	tabindex?: number
 }>(), {
-	newTab: false,
 	outlined: false,
 	tabindex: -1,
 })
diff --git a/core/src/components/AppMenu.vue b/core/src/components/AppMenu.vue
index a32e901b6f8..14e20908593 100644
--- a/core/src/components/AppMenu.vue
+++ b/core/src/components/AppMenu.vue
@@ -39,7 +39,6 @@
 						:key="item.id"
 						ref="items"
 						:app="item"
-						:newTab="openInNewTab"
 						:outlined="item.id === 'more-apps'"
 						:tabindex="i === focusedIndex ? 0 : -1" />
 				</div>
@@ -134,10 +133,6 @@ export default defineComponent({
 			return this.appList.find((app) => app.active)
 		},
 
-		openInNewTab(): boolean {
-			return this.currentApp?.id !== 'dashboard'
-		},
-
 		// Stable-ordered list that focusedIndex indexes into; adds "More apps" for admins.
 		gridItems(): INavigationEntry[] {
 			return this.isAdmin ? [...this.appList, this.moreAppsEntry] : [...this.appList]
diff --git a/core/src/tests/components/AppItem.spec.ts b/core/src/tests/components/AppItem.spec.ts
index b8630e34073..4212a11c5f5 100644
--- a/core/src/tests/components/AppItem.spec.ts
+++ b/core/src/tests/components/AppItem.spec.ts
@@ -43,12 +43,4 @@ describe('core: AppItem', () => {
 		const wrapper = mount(AppItem, { propsData: { app: makeApp({ active: true }) } })
 		expect(wrapper.attributes('aria-current')).toBe('page')
 	})
-
-	it('newTab prop renders target="_blank" rel="noopener noreferrer"', () => {
-		const wrapper = mount(AppItem, {
-			propsData: { app: makeApp(), newTab: true },
-		})
-		expect(wrapper.attributes('target')).toBe('_blank')
-		expect(wrapper.attributes('rel')).toBe('noopener noreferrer')
-	})
 })