@conduction/nextcloud-vue 2.112.0 → 2.113.0

package/dist/esm/components/CnIndexPage/CnIndexPage.vue2.js

@@ -27,8 +27,8 @@ import { createRefLabelResolver } from '../../composables/useRefLabels.js';
 import { useSavedViewsApi } from '../../composables/useSavedViewsApi.js';
 import { METADATA_COLUMNS } from '../../constants/metadata.js';
 import { useObjectStore } from '../../store/useObjectStore.js';
-import { routeHref, followItemActionLink, resolveItemActionLink } from '../../utils/actionLink.js';
-import { resolveRegisteredHandler, buildOnSuccessRoute } from '../../utils/actionsDispatcher.js';
+import { routeHref, headerActionAsLink, followItemActionLink, resolveItemActionLink } from '../../utils/actionLink.js';
+import { resolveRegisteredHandler, isExternalActionTarget, buildOnSuccessRoute } from '../../utils/actionsDispatcher.js';
 import { columnFieldsOf, refIdOf, buildValueStages, buildReferenceStages, columnKeyOfRow, resolveDropValue, fullTargetOf, referenceTargetOf, fetchReferenceObjects } from '../../utils/boardReferenceColumns.js';
 import { isColumnSortable } from '../../utils/columnFilters.js';
 import { reportBindingProblems } from '../../utils/diagnostics.js';
@@ -2589,11 +2589,16 @@ var script = {
 		 * (unless the handler is the `'none'` keyword) emits
 		 * `@header-action({ action: id, id })`.
 		 *
+		 * An entry with `href` instead renders as a real link and is never
+		 * dispatched or emitted: an absolute `http(s)`, `mailto:`, `tel:` or
+		 * `//host/...` URL is used as written, an instance path (leading `/`) goes through
+		 * `generateUrl()`. `target: '_blank'` opens it in a new tab.
+		 *
 		 * Reserved ids (those used by built-ins on the bar — `refresh`,
 		 * `import`, `export`, `copy`, `delete`) are dropped from the
 		 * merged list with a `console.warn`.
 		 *
-		 * @type {Array<{ id: string, label: string, icon?: string, handler?: string|(() => void), route?: string, disabled?: boolean }>}
+		 * @type {Array<{ id: string, label: string, icon?: string, handler?: string|(() => void), route?: string, href?: string, target?: ('_blank'|'_self'), disabled?: boolean }>}
 		 */
 		headerActions: {
 			type: Array,
@@ -3046,7 +3051,7 @@ var script = {
 		/**
 		 * The header actions that open a dialog: `type: "open-form"` (the
 		 * schema-driven create form, as the navigation's primary action opens
-		 * it) or `type: "open-modal"`, without a `handler` of their own.
+		 * it) or `type: "open-modal"`, without a `handler` or `href` of their own.
 		 * Reserved ids are left out, as in `mergedHeaderActions`.
 		 *
 		 * @spec openspec/changes/index-header-open-form/specs/index-header/spec.md#requirement-an-index-header-action-can-open-the-create-form
@@ -3058,9 +3063,18 @@ var script = {
 				&& typeof entry.id === 'string'
 				&& !reserved.has(entry.id)
 				&& (entry.handler === undefined || entry.handler === null || entry.handler === '')
+				&& !entry.href
 				&& (entry.type === 'open-form' || entry.type === 'open-modal'))
 		},
 
+		/**
+		 * The header actions without reserved ids, each through
+		 * `resolveHeaderHandler` (a manifest `href` entry becomes a link with
+		 * `_dispatchSuppress`, so the page neither runs nor emits it).
+		 *
+		 * @spec openspec/changes/header-action-href/specs/header-action-href/spec.md#requirement-a-header-action-with-href-is-a-link-that-runs-and-emits-nothing
+		 * @return {Array<object>}
+		 */
 		mergedHeaderActions() {
 			const reserved = new Set(['refresh', 'import', 'export', 'copy', 'delete']);
 			const merged = [];
@@ -6099,11 +6113,13 @@ var script = {
 		 *
 		 * @spec openspec/changes/zuiddrecht-pixel-gaps-3/specs/zuiddrecht-pixel-gaps-3/spec.md#requirement-an-index-page-can-take-the-board-header
 		 * @spec openspec/changes/screens-index-header-buttons-parity/specs/index-list-board-look/spec.md#requirement-the-add-header-button-carries-a-plus
-		 * @return {Array<{key: string, label: string, action: string, variant: string, icon: string, format: string}>}
+		 * @spec openspec/changes/header-action-href/specs/header-action-href/spec.md#requirement-a-header-action-with-href-is-a-link-that-runs-and-emits-nothing
+		 * @return {Array<{key: string, label: string, action: string, variant: string, icon: string, format: string, link: ({href: string, target: (string|undefined)}|null)}>}
 		 */
 		resolvedHeaderButtons() {
 			const headerButtons = this.presented('headerButtons');
 			const declared = Array.isArray(headerButtons) ? headerButtons : [];
+			const builtInActions = new Set(['add', 'export', 'import', 'refresh', 'save-view', 'actions-menu']);
 			return declared
 				.filter((button) => button && typeof button === 'object' && typeof button.action === 'string' && button.action !== '')
 				.map((button, index) => {
@@ -6130,6 +6146,13 @@ var script = {
 					if (label === '' && button.action === 'save-view') {
 						label = translate('nextcloud-vue', 'Save view');
 					}
+					const run = typeof button.run === 'string' ? button.run : '';
+					// A button that runs a manifest `href` header action is that link;
+					// a `type: "navigate"` or function-handler entry keeps dispatching.
+					const actionId = button.action === 'export' ? run : (builtInActions.has(button.action) ? '' : button.action);
+					const linked = actionId
+						? this.mergedHeaderActions.find((entry) => entry && entry.id === actionId && entry.href && entry._dispatchSuppress && (!entry.type || entry.type === 'handler'))
+						: null;
 					return {
 						key: String(button.id || button.action || index),
 						label,
@@ -6137,7 +6160,8 @@ var script = {
 						variant: button.variant === 'primary' ? 'primary' : 'secondary',
 						icon,
 						format: button.format === 'excel' ? 'excel' : 'csv',
-						run: typeof button.run === 'string' ? button.run : '',
+						run,
+						link: linked ? { href: linked.href, target: linked.linkTarget || undefined } : null,
 						items: Array.isArray(button.items)
 							? button.items.filter((item) => (typeof item === 'string' && item !== '')
 								|| (item && typeof item === 'object' && typeof item.heading === 'string' && item.heading !== ''))
@@ -7795,8 +7819,11 @@ var script = {
 		 * manifest-actions-dispatch — `navigate`, `emit`, `none`, or a
 		 * registry name (resolved through `resolveRegisteredHandler`).
 		 *
+		 * @spec openspec/changes/header-action-href/specs/header-action-href/spec.md#requirement-a-header-action-with-href-is-a-link-that-runs-and-emits-nothing
 		 * @param {object} entry Raw headerActions entry.
-		 * @return {object} Possibly-mutated copy: function-typed
+		 * @return {object} Possibly-mutated copy: an `href` becomes a
+		 *   resolved link with `linkTarget`, no handler (a function one
+		 *   included) and `_dispatchSuppress: true`; function-typed
 		 *   handlers untouched; `'emit'` and unknown registry names
 		 *   strip the `handler` (emit-only fall-through); `'navigate'`
 		 *   becomes a `$router.push` thunk (with the entry's literal
@@ -7808,14 +7835,23 @@ var script = {
 			if (!entry) {
 				return entry
 			}
+			// An `href` makes the entry a link, which only navigates: it
+			// carries no handler, and the page emits nothing for it.
+			if (typeof entry.href === 'string' && entry.href !== '') {
+				const link = headerActionAsLink(entry);
+				return link.href ? { ...link, linkTarget: link.target, _dispatchSuppress: true } : link
+			}
 			const handler = entry.handler;
 			if (typeof handler === 'function') {
 				return { ...entry }
 			}
 			// `type: "navigate"` with a `target` URL is a link: the bar and the
-			// header's Actions menu render it as a real anchor in a new tab.
+			// header's Actions menu render it as a real anchor, an external URL
+			// in a new tab.
 			if (typeof handler !== 'string' && entry.type === 'navigate' && typeof entry.target === 'string' && entry.target !== '' && !entry.href) {
-				return { ...entry, href: entry.target }
+				return isExternalActionTarget(entry.target)
+					? { ...entry, href: entry.target, linkTarget: '_blank' }
+					: { ...entry, href: entry.target }
 			}
 			if (typeof handler !== 'string') {
 				// No handler declared — emit-only fall-through.
@@ -7986,8 +8022,9 @@ var script = {
 		 *
 		 * @spec openspec/changes/screens-index-header-actions-menu/specs/index-header/spec.md#requirement-the-header-actions-menu-lists-what-its-items-name
 		 * @spec openspec/changes/screens-index-header-menu-headings/specs/index-header/spec.md#requirement-the-header-actions-menu-can-draw-group-headings-and-separators
+		 * @spec openspec/changes/header-action-href/specs/header-action-href/spec.md#requirement-a-header-action-with-href-is-a-link-that-runs-and-emits-nothing
 		 * @param {{items?: Array<(string|{heading: string})>|null, format?: string}} button The resolved header button.
-		 * @return {Array<{key: string, label: string, kind?: ('heading'|'separator'), icon?: string, href?: string, disabled?: boolean, title?: string, iconComponent?: object, run?: Function}>}
+		 * @return {Array<{key: string, label: string, kind?: ('heading'|'separator'), icon?: string, href?: string, linkTarget?: string, disabled?: boolean, title?: string, iconComponent?: object, run?: () => void}>}
 		 */
 		headerMenuEntries(button) {
 			const fromAction = (entry) => ({
@@ -7995,6 +8032,7 @@ var script = {
 				label: entry.label ? this.cnTranslate(entry.label) : '',
 				icon: entry.icon && typeof entry.icon === 'string' && !entry.icon.startsWith('icon-') ? entry.icon : '',
 				href: typeof entry.href === 'string' && entry.href !== '' ? entry.href : '',
+				linkTarget: entry.linkTarget || '',
 				// A `handler: "navigate"` entry whose route resolves carries `to`
 				// (resolveHeaderHandler drops its handler): the menu draws it as
 				// an in-app link, as the actions bar does, so it navigates.
@@ -8028,7 +8066,7 @@ var script = {
 				}),
 
 				documentation: () => (this.documentationUrl
-					? { key: 'documentation', label: this.documentationLabel || translate('nextcloud-vue', 'Documentation'), icon: 'BookOpenVariantOutline', href: this.documentationUrl }
+					? { key: 'documentation', label: this.documentationLabel || translate('nextcloud-vue', 'Documentation'), icon: 'BookOpenVariantOutline', href: this.documentationUrl, linkTarget: '_blank' }
 					: null),
 			};
 			const entries = [];
@@ -8124,13 +8162,18 @@ var script = {
 
 		/**
 		 * A header button was clicked: run its built-in action, or dispatch
-		 * it as the `headerActions` entry with that id.
+		 * it as the `headerActions` entry with that id. A button that is a
+		 * link is left to the browser.
 		 *
 		 * @spec openspec/changes/zuiddrecht-pixel-gaps-3/specs/zuiddrecht-pixel-gaps-3/spec.md#requirement-an-index-page-can-take-the-board-header
-		 * @param {{action: string, format: string}} button The resolved button.
+		 * @spec openspec/changes/header-action-href/specs/header-action-href/spec.md#requirement-a-header-action-with-href-is-a-link-that-runs-and-emits-nothing
+		 * @param {{action: string, format: string, link?: object}} button The resolved button.
 		 * @return {void}
 		 */
 		onHeaderButton(button) {
+			if (button.link) {
+				return
+			}
 			if (button.action === 'add') {
 				this.onAddClick();
 			} else if (button.action === 'export') {