@conduction/nextcloud-vue 2.56.0-beta.4 → 2.56.0-beta.5

package/src/components/CnDateAxisView/CnDateAxisView.vue

@@ -35,6 +35,7 @@
 						:style="barStyle(bar)"
 						:aria-label="barLabel(bar)"
 						@click="openRow(bar.row, $event)"
+						@mousedown="preventMiddleClickAutoscroll"
 						@auxclick="onAuxClick(bar.row, $event)">
 						{{ barText(bar) }}
 					</button>
@@ -61,6 +62,7 @@
 					:data-row-id="rowKeyOf(row)"
 					:aria-label="unplannedLabelFor(row)"
 					@click="openRow(row, $event)"
+					@mousedown="preventMiddleClickAutoscroll"
 					@auxclick="onAuxClick(row, $event)">
 					{{ nameOf(row) }}
 				</button>
@@ -72,7 +74,7 @@
 <script>
 import { translate as t } from '@nextcloud/l10n'
 import { buildDateAxisLanes } from '../../utils/dateAxisLanes.js'
-import { isRowMiddleClick } from '../../utils/rowAuxClick.js'
+import { isRowMiddleClick, preventMiddleClickAutoscroll } from '../../utils/rowAuxClick.js'
 
 /**
  * CnDateAxisView — the index page's rows on a time scale.
@@ -133,7 +135,7 @@ export default {
 		},
 	},
 
-	emits: ['row-click'],
+	emits: ['row-click', 'row-aux-click'],
 
 	computed: {
 		/** @return {object} The layout. */
@@ -176,23 +178,27 @@ export default {
 	methods: {
 		t,
 
+		preventMiddleClickAutoscroll,
+
 		/**
 		 * Open one record. Both the bar on the axis and the label of an
 		 * undated row come through here, so the two paths cannot drift.
 		 *
 		 * @param {object} row The record the reader picked.
-		 * @param {MouseEvent} [event] The click or middle-click event.
+		 * @param {MouseEvent} [event] The click event.
 		 * @return {void} Nothing.
 		 */
 		openRow(row, event) {
 			/**
-			 * @event row-click Emitted when a reader opens a record, from its bar on the axis or from the label of an undated row, on a click or a middle click. Payload: `(row, event)` — the row and the native click/auxclick event, so the host can open it in a new tab on a ctrl/cmd/shift or middle click.
+			 * @event row-click Emitted when a reader opens a record, from its bar on the axis or from the label of an undated row. Payload: `(row, event)` — the row and the native click event, so the host can open it in a new tab on a ctrl/cmd/shift click. A middle click emits `row-aux-click` instead.
 			 */
 			this.$emit('row-click', row, event)
 		},
 
 		/**
-		 * Middle click on a bar: open the record like a click would.
+		 * Middle click on a bar or an undated row's label: emits
+		 * `row-aux-click`, not `row-click`, so an existing listener that
+		 * navigates never moves the current tab away.
 		 *
 		 * @param {object} row The record the reader picked.
 		 * @param {MouseEvent} event The auxclick event.
@@ -200,7 +206,10 @@ export default {
 		 */
 		onAuxClick(row, event) {
 			if (isRowMiddleClick(event)) {
-				this.openRow(row, event)
+				/**
+				 * @event row-aux-click Emitted when a reader middle-clicks a record's bar or undated-row label, for opening it in a new tab (see `openRowTarget`). Payload: `(row, event)` — the row and the native auxclick event.
+				 */
+				this.$emit('row-aux-click', row, event)
 			}
 		},
 

package/src/components/CnDetailPage/CnDetailPage.md

@@ -394,7 +394,7 @@ CnDetailPage consumes the same `actions[].handler` contract as CnIndexPage. When
 | `relatedCollections` | Array | `[]` | Declarative related-object list sections below the body; each `{ title?, register, schema, filter?, columns?, sort?, limit?, rowRoute? }` renders a `CnObjectListWidget` scoped to this object via `@objectId`. Emits `@related-row-click`. See docs `cn-related-collections.md`. |
 | `summaryAggregates` | Array | `[]` | Declarative cross-schema count/sum/avg chips in the header; each `{ label, register, schema, metric?, field?, filter?, format? }` scoped to this object. See docs `cn-summary-aggregates.md`. |
 | `relationLinks` | Array | `[]` | Declarative relation-link actions; each `{ label?, register, schema, fkField, labelField?, allowCreate?, title?, selectLabel? }` opens a search-and-link modal that patches a FK on this object. Emits `@relation-linked`. See docs `cn-relation-link-modal.md`. |
-| `bodyWidgets` | Array | `[]` | Declarative IN-BODY sections; each `{ id?, component, title?, props?, placement?, colSpan? }` renders a registered host-app component as a titled body section (NOT the sidebar) with token-resolved props (`@objectId` / `@object.<field>` / `@workspace.<key>`) and the object context provided on `cnSectionContext`. `placement` = `before-body`/`after-data`/`after-related`/`end`. No sidebar tab required. See docs `cn-body-sections.md`. |
+| `bodyWidgets` | Array | `[]` | Declarative IN-BODY sections; each `{ id?, component, title?, props?, placement?, colSpan? }` renders a registered host-app component as a titled body section (NOT the sidebar) with token-resolved props (`@objectId` / `@object.<field>` / `@workspace.<key>`) and the object context provided on `cnSectionContext`, whose `setObject(object)` shows an object a section saved without a re-fetch. `placement` = `before-body`/`after-data`/`after-related`/`end`. No sidebar tab required. See docs `cn-body-sections.md`. |
 
 ## Header actions menu
 

package/src/components/CnDetailPage/CnDetailPage.vue

@@ -2885,8 +2885,9 @@ export default {
 		 * (`@objectId` / `@object.<field>` / `@workspace.<key>`) AND provided on
 		 * `cnSectionContext` for host section components that inject. Mirrors the
 		 * `cnObjectContext` ref the abstract list/stat widgets already read.
+		 * `setObject(object)` shows an object a section saved without a re-fetch.
 		 *
-		 * @return {{objectId: (string|null), object: (object|null), register: string, schema: string}}
+		 * @return {{objectId: (string|null), object: (object|null), register: string, schema: string, setObject: function(object): void}}
 		 */
 		sectionContext() {
 			const resolved = this.resolvedSidebar || {}
@@ -2896,6 +2897,7 @@ export default {
 				register: resolved.register || this.register || this.sidebarProps?.register || '',
 				schema: resolved.schema || this.schema || this.resolvedObjectType || this.sidebarProps?.schema || '',
 				config: this.cnAppConfigRef,
+				setObject: (object) => this.setCurrentObject(object),
 			}
 		},
 	},
@@ -3392,6 +3394,32 @@ export default {
 			this.$emit('related-row-click', payload)
 		},
 
+		/**
+		 * Replace the page's object in the store cache, for a section that saved
+		 * it and holds the saved version. An object with another id is ignored.
+		 *
+		 * @param {object} object The saved object.
+		 */
+		setCurrentObject(object) {
+			const store = this.effectiveObjectStore
+			const type = this.resolvedObjectType
+			if (!store || !type || !this.objectId || !object) {
+				// eslint-disable-next-line no-console -- tells the integrating section its save is not shown
+				console.warn('[CnDetailPage] setObject ignored: the page has no object store, type or id')
+				return
+			}
+			const id = object.id ?? object['@self']?.id
+			if (id !== undefined && id !== null && String(id) !== String(this.objectId)) {
+				// eslint-disable-next-line no-console -- tells the integrating section its save is not shown
+				console.warn(`[CnDetailPage] setObject ignored: object ${id} is not the page's object ${this.objectId}`)
+				return
+			}
+			store.objects = {
+				...store.objects,
+				[type]: { ...(store.objects?.[type] || {}), [this.objectId]: object },
+			}
+		},
+
 		/**
 		 * Schema-driven fetch entry point — no-op outside the
 		 * `register`+`schema`+`objectId` mode. Registers the type on

package/src/components/CnFeaturesAndRoadmapView/CnFeaturesAndRoadmapView.vue

@@ -29,8 +29,7 @@
 						v-if="resolvedSuggestUrl"
 						variant="primary"
 						:href="resolvedSuggestUrl"
-						target="_blank"
-						rel="noopener noreferrer">
+						target="_blank">
 						<template #icon>
 							<Plus :size="20" />
 						</template>

package/src/components/CnFilesBrowser/CnFilesBrowser.vue

@@ -180,6 +180,7 @@
 					data-testid="cn-files-browser-row"
 					:data-name="node.basename"
 					@click="onRowClick(node, $event)"
+					@mousedown="preventMiddleClickAutoscroll"
 					@auxclick="onRowAuxClick(node, $event)">
 					<td class="cn-files-browser__col-icon">
 						<img
@@ -258,7 +259,6 @@
 								v-if="node.fileid"
 								:href="permalink(node)"
 								target="_blank"
-								rel="noopener noreferrer"
 								:closeAfterClick="true">
 								<template #icon>
 									<OpenInNew :size="20" />
@@ -278,6 +278,7 @@
 					data-testid="cn-files-browser-linked-row"
 					:data-name="item.name"
 					@click="openLinked(item)"
+					@mousedown="preventMiddleClickAutoscroll"
 					@auxclick="onLinkedAuxClick(item, $event)">
 					<td class="cn-files-browser__col-icon">
 						<img
@@ -316,7 +317,6 @@
 								v-if="item.href"
 								:href="item.href"
 								target="_blank"
-								rel="noopener noreferrer"
 								:closeAfterClick="true"
 								data-testid="cn-files-browser-linked-open">
 								<template #icon>
@@ -473,7 +473,7 @@ import CnCellRenderer from '../CnCellRenderer/CnCellRenderer.vue'
 import CnIcon from '../CnIcon/CnIcon.vue'
 import { readUserPreference, writeUserPreference } from '../../composables/useUserPreferences.js'
 import { dispatchAction } from '../../utils/actionsDispatcher.js'
-import { isNewTabClick, isRowMiddleClick } from '../../utils/rowAuxClick.js'
+import { isNewTabClick, isRowMiddleClick, preventMiddleClickAutoscroll } from '../../utils/rowAuxClick.js'
 import { ACTIONS_NEEDING_THE_FILES_PAGE, crumbsFor, joinPath } from './filesBrowser.js'
 import {
 	attributePropertiesFor,
@@ -898,6 +898,8 @@ export default {
 	methods: {
 		t,
 
+		preventMiddleClickAutoscroll,
+
 		/**
 		 * Hold the file input element from its function ref.
 		 *

package/src/components/CnFilesWidget/CnFilesWidget.vue

@@ -725,18 +725,14 @@ export default {
 		},
 
 		/**
-		 * Handle a row click — descend into folders, open files in the Files app.
+		 * Handle a folder row click by descending into it. A file row is a
+		 * real link to the Files app and never reaches this.
 		 *
-		 * @param {object} item the clicked item.
+		 * @param {object} item the clicked folder item.
 		 * @return {void}
 		 */
 		onItemClick(item) {
-			if (item.isFolder) {
-				const next = this.joinPath(this.currentSubPath, item.name)
-				this.navigateTo(next)
-			} else {
-				this.openFileInFilesApp(item.fileId)
-			}
+			this.navigateTo(this.joinPath(this.currentSubPath, item.name))
 		},
 
 		/**
@@ -779,19 +775,6 @@ export default {
 			return `${trimmedBase}/${trimmedName}`
 		},
 
-		/**
-		 * Deep-link a file into the Nextcloud Files app in a new tab.
-		 *
-		 * @param {number|string} fileId the Nextcloud file id.
-		 * @return {void}
-		 */
-		openFileInFilesApp(fileId) {
-			const url = this.fileUrl(fileId)
-			if (url) {
-				window.open(url, '_blank', 'noopener,noreferrer')
-			}
-		},
-
 		/**
 		 * The Files-app permalink for a file.
 		 *

package/src/components/CnFlowRunsWidget/CnFlowRunsWidget.vue

@@ -31,15 +31,15 @@
 			</p>
 			<ul v-else class="cn-flow-runs-widget__list">
 				<li
-					v-for="run in rows"
-					:key="run.uuid"
+					v-for="(run, index) in rows"
+					:key="run.uuid || `row-${index}`"
 					class="cn-flow-runs-widget__row"
 					:class="{ 'cn-flow-runs-widget__row--linked': isLinked }"
 					:data-status="run.status">
 					<component
-						:is="runHrefs[run.uuid] ? 'a' : 'div'"
+						:is="runHref(run) ? 'a' : 'div'"
 						class="cn-flow-runs-widget__row-content"
-						:href="runHrefs[run.uuid] || undefined"
+						:href="runHref(run) || undefined"
 						@click="onRowClick($event, run)">
 						<span
 							class="cn-flow-runs-widget__dot"
@@ -70,15 +70,15 @@
 				</p>
 				<ul v-else class="cn-flow-runs-widget__list">
 					<li
-						v-for="run in completedRows"
-						:key="run.uuid"
+						v-for="(run, index) in completedRows"
+						:key="run.uuid || `completed-${index}`"
 						class="cn-flow-runs-widget__row cn-flow-runs-widget__row--terminal"
 						:class="{ 'cn-flow-runs-widget__row--linked': isLinked }"
 						:data-status="run.status">
 						<component
-							:is="runHrefs[run.uuid] ? 'a' : 'div'"
+							:is="runHref(run) ? 'a' : 'div'"
 							class="cn-flow-runs-widget__row-content"
-							:href="runHrefs[run.uuid] || undefined"
+							:href="runHref(run) || undefined"
 							@click="onRowClick($event, run)">
 							<span
 								class="cn-flow-runs-widget__dot cn-flow-runs-widget__dot--terminal"
@@ -362,19 +362,6 @@ export default {
 	},
 
 	computed: {
-		/**
-		 * Each run's link href by uuid, resolved once per render.
-		 *
-		 * @return {{[uuid: string]: string}}
-		 */
-		runHrefs() {
-			const out = {}
-			for (const run of [...this.rows, ...this.completedRows]) {
-				out[run.uuid] = this.runHref(run)
-			}
-			return out
-		},
-
 		/**
 		 * Effective translate function: the explicit prop, then the injected
 		 * host one, then `@nextcloud/l10n` under the library's own app id.

package/src/components/CnFlowsPage/CnFlowsPage.vue

@@ -13,7 +13,8 @@
 		:showEditAction="false"
 		:actions="rowActions"
 		rowClickToView
-		@rowClick="openFlow">
+		@rowClick="openFlow"
+		@rowAuxClick="openFlow">
 		<template #header-actions>
 			<NcButton variant="primary" :to="newFlowRoute">
 				<template #icon>

package/src/components/CnIndexPage/CnIndexPage.vue

@@ -451,6 +451,7 @@
 					@sort="onSortEvent"
 					@select="onSelect"
 					@rowClick="onRowClick"
+					@rowAuxClick="onRowAuxClick"
 					@rowContextMenu="onRowContextMenu">
 					<!-- Pass through column slots -->
 					<template
@@ -548,6 +549,7 @@
 					:runTransition="runTransition"
 					:paged="isPaged"
 					@cardClick="onRowClick"
+					@cardAuxClick="onRowAuxClick"
 					@moved="onBoardMoved" />
 
 				<!--
@@ -563,7 +565,8 @@
 					:laneField="dateAxis.laneField || ''"
 					:labelField="dateAxis.labelField || ''"
 					:rowKey="rowKey"
-					@rowClick="onRowClick" />
+					@rowClick="onRowClick"
+					@rowAuxClick="onRowAuxClick" />
 
 				<!-- List view -->
 				<CnObjectList
@@ -576,6 +579,7 @@
 					:rowKey="rowKey"
 					:emptyText="emptyText"
 					@click="onRowClick"
+					@auxClick="onRowAuxClick"
 					@select="onSelect">
 					<!--
 						List-item slot resolution priority (highest first):
@@ -596,6 +600,7 @@
 							:register="register"
 							:selected="selected"
 							@click="(...args) => onRowClick(object, args.find(isDomEvent))"
+							@mousedown="onCustomItemMouseDown"
 							@auxclick="onCustomItemAuxClick(object, $event)"
 							@select="onSelect(toggleIdInArray(internalSelectedIds, object[rowKey]))" />
 					</template>
@@ -629,6 +634,7 @@
 					:rowKey="rowKey"
 					:emptyText="emptyText"
 					@click="onRowClick"
+					@auxClick="onRowAuxClick"
 					@select="onSelect">
 					<!--
 						Card slot resolution priority (highest first):
@@ -649,6 +655,7 @@
 							:register="register"
 							:selected="selected"
 							@click="(...args) => onRowClick(object, args.find(isDomEvent))"
+							@mousedown="onCustomItemMouseDown"
 							@auxclick="onCustomItemAuxClick(object, $event)"
 							@select="onSelect(toggleIdInArray(internalSelectedIds, object[rowKey]))" />
 					</template>
@@ -781,7 +788,7 @@ import { LIST_SHORTCUTS, listPaletteCommands, shortcutFor } from '../../utils/li
 import { multiKeySort } from '../../utils/multiKeySort.js'
 import { resolveDeepTokens, resolveFilterValue } from '../../utils/resolveFilterTokens.js'
 import { availableRowActions, DEFAULT_ROW_ACTION_FIELD, refusalReasonFor, undeclaredRowActions } from '../../utils/rowActionAvailability.js'
-import { isNewTabClick, isRowMiddleClick, markNewTabHandled } from '../../utils/rowAuxClick.js'
+import { isNewTabClick, isNewTabHandled, isRowMiddleClick, markNewTabHandled, preventMiddleClickAutoscroll } from '../../utils/rowAuxClick.js'
 import { DEFAULT_ROW_INDICATOR_CAP } from '../../utils/rowIndicators.js'
 import { buildRouteQueryFromViewState, buildViewCreatePayload, extractViewState, extractViewStateFromRouteQuery, savedViewScope, viewMatchesScope } from '../../utils/savedViewHelpers.js'
 import { columnsFromSchema } from '../../utils/schema.js'
@@ -995,7 +1002,8 @@ function namedFilterToQuery(values) {
  * @event {object} mass-export — Mass export confirmed. Payload: { ids, format }
  * @event {object} mass-import — Mass import confirmed. Payload: import data
  * @event {void} refresh — Refresh button clicked
- * @event {object} row-click — Table row or card clicked or middle-clicked. Payload: (row, event) — the row object and the native click/auxclick event
+ * @event {object} row-click — Table row or card clicked. Payload: (row, event) — the row object and the native click event
+ * @event {object} row-aux-click — Table row or card middle-clicked. Payload: (row, event) — the row object and the native auxclick event
  * @event {{ key: string, order: string }} sort — Column sort changed
  * @event {number} page-changed — Pagination page changed
  * @event {number} page-size-changed — Pagination page size changed
@@ -2568,6 +2576,7 @@ export default {
 		'quick-filter-change',
 		'refresh',
 		'row-click',
+		'row-aux-click',
 		'search',
 		'select',
 		'sort',
@@ -2810,6 +2819,9 @@ export default {
 		 * that is what Add does (no host `@add` listener) and the router can
 		 * resolve it. The bar then renders Add as a real link; null otherwise.
 		 *
+		 * `$.vnode.props` is not reactive, so an `add` listener attached or
+		 * removed after mount does not re-evaluate this.
+		 *
 		 * @return {string|object|null}
 		 */
 		addLinkTo() {
@@ -5565,21 +5577,60 @@ export default {
 		/**
 		 * Row/card click: toggles selection when `selectable` (covers the custom
 		 * `cardComponent` path), otherwise emits `row-click` for navigation. A
-		 * ctrl/cmd/shift or middle click opens a named source's row in a new tab.
+		 * ctrl/cmd/shift click opens a named source's row in a new tab.
 		 *
 		 * The `@event` block below sits directly against its `$emit`, and must
 		 * stay there. vue-docgen binds an event's description to the emit it
 		 * IMMEDIATELY precedes.
 		 *
 		 * @param {object} row The clicked row object
-		 * @param {MouseEvent|KeyboardEvent} [event] The originating click/auxclick event.
+		 * @param {MouseEvent|KeyboardEvent} [event] The originating click event.
 		 */
 		onRowClick(row, event) {
+			if (!this.routeClickedRow(row, event)) {
+				return
+			}
+			/**
+			 * @event row-click Emitted on a row/card click for navigation. Fires when `selectable` is false, OR when `rowClickToView` is set and something can open the row (selection then happens via the checkbox). Payload: `(row, event)` — the clicked row and the native click event (absent for a keyboard shortcut or map marker), so a host can open the row in a new tab on a ctrl/cmd/shift click with `openRowTarget`. A middle click emits `row-aux-click` instead.
+			 * @type {object} The clicked row object.
+			 */
+			this.$emit('row-click', ...(event ? [row, event] : [row]))
+		},
+
+		/**
+		 * Row/card middle click: opens a named source's row in a new tab and
+		 * emits `row-aux-click`, not `row-click`, so a host whose `row-click`
+		 * listener navigates never moves the current tab away.
+		 *
+		 * @param {object} row The clicked row object
+		 * @param {MouseEvent} event The originating auxclick event.
+		 */
+		onRowAuxClick(row, event) {
+			if (!this.routeClickedRow(row, event)) {
+				return
+			}
+			/**
+			 * @event row-aux-click Emitted on a row/card middle click, under the same conditions as `row-click`, so a host can open the row in a new tab with `openRowTarget`. Payload: `(row, event)` — the clicked row and the native auxclick event.
+			 * @type {object} The clicked row object.
+			 */
+			this.$emit('row-aux-click', row, event)
+		},
+
+		/**
+		 * The shared part of a row click and a middle click: on a selectable
+		 * page a plain click toggles selection instead, and a named source opens
+		 * its own row.
+		 *
+		 * @param {object} row The clicked row object
+		 * @param {MouseEvent|KeyboardEvent} [event] The originating event.
+		 * @return {boolean} Whether the click navigates, so the caller emits.
+		 */
+		routeClickedRow(row, event) {
 			if (this.selectable && !this.rowClickOpens) {
 				if (!event || event.button === undefined || event.button === 0) {
 					this.onSelect(this.toggleIdInArray(this.internalSelectedIds, row[this.rowKey]))
 				}
-				return
+				return false
 			}
 			// A named source knows where its rows live. Emitting only would leave
 			// the click inert on a manifest page, which has no listener to bind —
@@ -5626,11 +5677,7 @@ export default {
 					}
 				}
 			}
-			/**
-			 * @event row-click Emitted on a row/card click (or middle click) for navigation. Fires when `selectable` is false, OR when `rowClickToView` is set and something can open the row (selection then happens via the checkbox). Payload: `(row, event)` — the clicked row and the native click/auxclick event (absent for a keyboard shortcut or map marker), so a host can open the row in a new tab on a ctrl/cmd/shift or middle click with `openRowTarget`.
-			 * @type {object} The clicked row object.
-			 */
-			this.$emit('row-click', ...(event ? [row, event] : [row]))
+			return true
 		},
 
 		/**
@@ -5644,17 +5691,31 @@ export default {
 			return typeof Event !== 'undefined' && value instanceof Event
 		},
 
+		/**
+		 * Mousedown on a custom list/card component's root: when a middle click
+		 * opens the row, keep the browser from starting autoscroll.
+		 *
+		 * @param {MouseEvent} event The mousedown event.
+		 * @return {void}
+		 */
+		onCustomItemMouseDown(event) {
+			if (!this.selectable || this.rowClickOpens) {
+				preventMiddleClickAutoscroll(event)
+			}
+		},
+
 		/**
 		 * Middle click on a custom list/card component's root: open the row
-		 * like a click would.
+		 * like a middle click on a built-in card would. Skipped when something
+		 * inside the component already opened it in a new tab.
 		 *
 		 * @param {object} row The component's row.
 		 * @param {MouseEvent} event The auxclick event.
 		 * @return {void}
 		 */
 		onCustomItemAuxClick(row, event) {
-			if (isRowMiddleClick(event)) {
-				this.onRowClick(row, event)
+			if (isRowMiddleClick(event) && !isNewTabHandled(event)) {
+				this.onRowAuxClick(row, event)
 			}
 		},
 

package/src/components/CnIndexPage/manifestActionDispatch.js

@@ -37,19 +37,39 @@ function resolveRowToken(value, row) {
 }
 
 /**
- * The route params a `handler: "navigate"` action pushes for a row: the row id,
- * overridden by the declared `params` after the `{field}` token grammar. A
- * token naming a field the row lacks is dropped.
+ * The implicit `{ id }` param for a row, only when the named route declares
+ * `:id`: vue-router warns about (and drops) a param its path does not take.
+ * Without a router to ask, or for a route it does not know, the id is kept.
  *
- * @param {object} action Manifest action descriptor.
+ * @param {object} [router] The app's router.
+ * @param {string} name The route name.
  * @param {object} row The row.
  * @param {string} rowKey The row's id field.
+ * @return {object} `{ id }`, or an empty object.
+ */
+function rowIdParam(router, name, row, rowKey) {
+	const routes = typeof router?.getRoutes === 'function' ? router.getRoutes() : null
+	const route = routes ? routes.find((r) => r.name === name) : null
+	if (route && !/:id\b/.test(route.path)) {
+		return {}
+	}
+	return { id: row?.[rowKey] }
+}
+
+/**
+ * The route params a `handler: "navigate"` action pushes for a row: the row id
+ * (when the route takes one), overridden by the declared `params` after the
+ * `{field}` token grammar. A token naming a field the row lacks is dropped.
+ *
+ * @param {object} action Manifest action descriptor.
+ * @param {object} row The row.
+ * @param {object} ctx Dispatch context (`rowKey`, `router`).
  * @param {boolean} warn Whether to warn about a dropped param.
  * @return {object} The params.
  */
-function navigateParams(action, row, rowKey, warn) {
+function navigateParams(action, row, ctx, warn) {
 	const declaredParams = (action.params && typeof action.params === 'object') ? action.params : null
-	const params = { id: row?.[rowKey] }
+	const params = rowIdParam(ctx.router, action.route, row, ctx.rowKey)
 	for (const [key, declared] of Object.entries(declaredParams || {})) {
 		const { resolved, value } = resolveRowToken(declared, row)
 		if (resolved) {
@@ -88,7 +108,7 @@ export function resolveActionTarget(action, row, ctx, warn = false) {
 			return null
 		}
 		if (type === 'open-page') {
-			return { target: { name: target, params: { id: row?.[ctx.rowKey] } }, external: false }
+			return { target: { name: target, params: rowIdParam(ctx.router, target, row, ctx.rowKey) }, external: false }
 		}
 		return { target, external: isExternalActionTarget(target) }
 	}
@@ -96,7 +116,7 @@ export function resolveActionTarget(action, row, ctx, warn = false) {
 		if (typeof action.route !== 'string' || action.route.length === 0) {
 			return null
 		}
-		return { target: { name: action.route, params: navigateParams(action, row, ctx.rowKey, warn) }, external: false }
+		return { target: { name: action.route, params: navigateParams(action, row, ctx, warn) }, external: false }
 	}
 	return null
 }
@@ -115,8 +135,8 @@ export function resolveActionTarget(action, row, ctx, warn = false) {
  *   - `open-modal` → not wired for index actions; falls back to `@action`.
  *
  * Handler dispatch (`type: 'handler'`, the default) reads `action.handler`:
- *   - `navigate` → $router.push to `action.route` with `{ id: row[rowKey] }`,
- *     merged with the `action.params` map (declared params win — so a
+ *   - `navigate` → $router.push to `action.route` with `{ id: row[rowKey] }`
+ *     (when the route declares `:id`), merged with the `action.params` map (declared params win — so a
  *     "New X" action can navigate to a detail route with `{ id: "new" }`).
  *     Param strings run the `{field}` row-token grammar: `"{id}"` resolves to
  *     `row.id` (type preserved), `"item-{id}"` interpolates, and a brace-less
@@ -187,7 +207,7 @@ export function resolveActionHandler(action, ctx) {
 		// `{field}` row-token grammar first — an unresolved token is dropped
 		// rather than pushed as a literal `%7Bid%7D` path segment.
 		return (row) => {
-			ctx.router.push({ name: route, params: navigateParams(action, row, ctx.rowKey, true) })
+			ctx.router.push({ name: route, params: navigateParams(action, row, ctx, true) })
 		}
 	}
 

package/src/components/CnIntegrationWidget/CnIntegrationWidgetEmpty.vue

@@ -25,8 +25,7 @@
 			<NcButton
 				variant="primary"
 				:href="docsUrl"
-				target="_blank"
-				rel="noopener">
+				target="_blank">
 				<template #icon>
 					<OpenInNew :size="18" />
 				</template>

package/src/components/CnInteractionFormWidget/CnInteractionFormWidget.vue

@@ -38,7 +38,6 @@
 				variant="tertiary"
 				:href="savedHref"
 				target="_blank"
-				rel="noopener noreferrer"
 				data-testid="cn-interaction-form-open">
 				<template #icon>
 					<OpenInNew :size="20" />

package/src/components/CnLinkButtonWidget/CnLinkButtonWidget.vue

@@ -420,10 +420,9 @@ export default {
 		 * @return {void}
 		 */
 		dispatchAction({ actionType, url, value }) {
+			// An `external` entry with a URL is a real link (see linkAttrsFor);
+			// one without a URL has nothing to open.
 			switch (actionType) {
-				case ACTION_TYPES.EXTERNAL:
-					this.handleExternal(url)
-					break
 				case ACTION_TYPES.INTERNAL:
 					this.$emit('internal-action', url)
 					break
@@ -434,20 +433,6 @@ export default {
 					break
 			}
 		},
-
-		/**
-		 * Open an external URL in a new tab; `noopener,noreferrer` prevents
-		 * the opened page reaching back via `window.opener`.
-		 *
-		 * @param {string} url the URL to open.
-		 * @return {void}
-		 */
-		handleExternal(url) {
-			if (typeof url !== 'string' || url === '') {
-				return
-			}
-			window.open(url, '_blank', 'noopener,noreferrer')
-		},
 	},
 }
 </script>