Allow target="_blank" in app navigation items

Which was removed in the Vue rewrite in #33728. This breaks things like nextcloud/external#79 Signed-off-by: Thomas Citharel <[email protected]>
nextcloud · Jan 4, 2023 · ef2ccfc · ef2ccfc
1 parent e235c64
commit ef2ccfc
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/core/src/components/AppMenu.vue b/core/src/components/AppMenu.vue
@@ -31,7 +31,9 @@
 					:class="{ 'has-unread': app.unread > 0 }"
 					:aria-label="appLabel(app)"
 					:title="app.name"
-					:aria-current="app.active ? 'page' : false">
+					:aria-current="app.active ? 'page' : false"
+					:target="app.target ? '_blank' : undefined"
+					:rel="app.target ? 'noopener noreferrer' : undefined">
 					<img :src="app.icon" alt="">
 					<div class="app-menu-entry--label">
 						{{ app.name }}

diff --git a/dist/core-main.js b/dist/core-main.js
diff --git a/dist/core-main.js.map b/dist/core-main.js.map