Merge pull request #7475 from nextcloud/backport/7474/stable30

[stable30] Several link bubble/preview fixes

Author: mejo-

cypress/e2e/nodes/Links.spec.js

@@ -47,7 +47,7 @@ describe('test link marks', function() {
 			cy.get('.link-view-bubble .widget-default', { timeout: 10000 })
 				.find('.widget-default--name')
 				.contains('Nextcloud')
-				.click({ force: true })
+				.click()
 		})
 
 		it('shows a link preview in the bubble after browsing to link', () => {
@@ -64,6 +64,16 @@ describe('test link marks', function() {
 				.contains('Nextcloud')
 		})
 
+		it('open button opens a new tab', () => {
+			const link = 'https://nextcloud.com/'
+			cy.insertLine(link)
+			clickLink(link)
+
+			cy.get('.link-view-bubble button[title="Open link"]').click()
+
+			cy.get('@winOpen').should('have.been.calledOnce')
+		})
+
 		it('closes the link bubble when clicking elsewhere', () => {
 			const link = 'https://nextcloud.com/'
 			cy.insertLine(link)

cypress/e2e/nodes/PreviewOptions.spec.js

@@ -23,6 +23,7 @@ describe('Preview Options', function() {
 	})
 
 	it('should render previewOptions correctly', function() {
+		cy.get('.action-button__text').contains('Open in new tab').should('be.visible')
 		cy.get('.action-button__text').contains('Remove link').should('be.visible')
 		cy.get('.action-radio__label').each(el => {
 			cy.wrap(el).invoke('text').should('match', /Text only|Show link preview/)

src/components/Editor.provider.js

@@ -3,6 +3,7 @@
  * SPDX-License-Identifier: AGPL-3.0-or-later
  */
 
+import { openLink } from '../helpers/links.js'
 import { logger } from '../helpers/logger.js'
 
 export const EDITOR = Symbol('tiptap:editor')
@@ -16,6 +17,7 @@ export const SYNC_SERVICE = Symbol('sync:service')
 export const EDITOR_UPLOAD = Symbol('editor:upload')
 export const HOOK_MENTION_SEARCH = Symbol('hook:mention-search')
 export const HOOK_MENTION_INSERT = Symbol('hook:mention-insert')
+export const OPEN_LINK_HANDLER = Symbol('editor:open-link-handler')
 
 export const useEditorMixin = {
 	inject: {
@@ -99,3 +101,13 @@ export const useMentionHook = {
 		},
 	},
 }
+export const useOpenLinkHandler = {
+	inject: {
+		$openLinkHandler: {
+			from: OPEN_LINK_HANDLER,
+			default: {
+				openLink,
+			},
+		},
+	},
+}

src/components/Editor/PreviewOptions.vue

@@ -27,11 +27,17 @@
 				{{ t('text', 'Show link preview') }}
 			</NcActionRadio>
 			<NcActionSeparator />
-			<NcActionButton close-after-click="true" @click="deleteNode">
+			<NcActionButton v-if="href" close-after-click @click="openLink">
+				<template #icon>
+					<OpenIcon :size="20" />
+				</template>
+				{{ t('text', 'Open in new tab') }}
+			</NcActionButton>
+			<NcActionButton close-after-click @click="deleteNode">
 				<template #icon>
 					<DeleteIcon :size="20" />
 				</template>
-				{{ t('text','Remove link') }}
+				{{ t('text', 'Remove link') }}
 			</NcActionButton>
 		</NcActions>
 	</div>
@@ -45,6 +51,7 @@ import NcActionCaption from '@nextcloud/vue/components/NcActionCaption'
 import NcActionSeparator from '@nextcloud/vue/components/NcActionSeparator'
 import DotsVerticalIcon from 'vue-material-design-icons/DotsVertical.vue'
 import DeleteIcon from 'vue-material-design-icons/Delete.vue'
+import OpenIcon from 'vue-material-design-icons/OpenInNew.vue'
 
 export default {
 	name: 'PreviewOptions',
@@ -57,13 +64,19 @@ export default {
 		NcActionRadio,
 		NcActionSeparator,
 		DeleteIcon,
+		OpenIcon,
 	},
 
 	props: {
 		type: {
 			type: String,
 			required: true,
 		},
+		href: {
+			type: String,
+			required: false,
+			default: '',
+		},
 		offset: {
 			type: Number,
 			required: true,
@@ -104,6 +117,10 @@ export default {
 				to: this.offset + this.nodeSize,
 			})
 		},
+		openLink() {
+			if (!this.href) return
+			window.open(this.href, '_blank').focus()
+		},
 	},
 }
 </script>

src/components/Link/LinkBubbleView.vue

@@ -10,6 +10,15 @@
 			<div class="link-view-bubble__title">
 				{{ title }}
 			</div>
+			<!-- open link -->
+			<NcButton :title="t('text', 'Open link')"
+				:aria-label="t('text', 'Open link')"
+				type="tertiary"
+				@click="openLink(href)">
+				<template #icon>
+					<OpenInNewIcon :size="20" />
+				</template>
+			</NcButton>
 			<!-- copy link -->
 			<NcButton :title="copyLinkTooltip"
 				:aria-label="copyLinkTooltip"
@@ -95,9 +104,11 @@ import CheckIcon from 'vue-material-design-icons/Check.vue'
 import CloseIcon from 'vue-material-design-icons/Close.vue'
 import ContentCopyIcon from 'vue-material-design-icons/ContentCopy.vue'
 import LinkOffIcon from 'vue-material-design-icons/LinkOff.vue'
+import OpenInNewIcon from 'vue-material-design-icons/OpenInNew.vue'
 import PencilIcon from 'vue-material-design-icons/Pencil.vue'
 
 import CopyToClipboardMixin from '../../mixins/CopyToClipboardMixin.js'
+import { useOpenLinkHandler } from '../Editor.provider.js'
 
 const PROTOCOLS_WITH_PREVIEW = ['http:', 'https:']
 
@@ -113,11 +124,13 @@ export default {
 		NcReferenceList,
 		NcTextField,
 		LinkOffIcon,
+		OpenInNewIcon,
 		PencilIcon,
 	},
 
 	mixins: [
 		CopyToClipboardMixin,
+		useOpenLinkHandler,
 	],
 
 	props: {
@@ -195,6 +208,10 @@ export default {
 			this.referenceTitle = null
 		},
 
+		openLink(href) {
+			this.$openLinkHandler.openLink(href)
+		},
+
 		async copyLink() {
 			await this.copyToClipboard(this.href)
 		},

src/css/prosemirror.scss

@@ -177,11 +177,6 @@ div.ProseMirror {
 		font-size: var(--default-font-size);
 	}
 
-	img {
-		cursor: default;
-		max-width: 100%;
-	}
-
 	hr {
 		padding: 2px 0;
 		border: none;
@@ -402,5 +397,6 @@ div.ProseMirror {
 }
 
 .tippy-content div {
+	box-sizing: border-box;
 	visibility: visible !important;
 }

src/editor.js

@@ -6,8 +6,9 @@
 import Vue from 'vue'
 import store from './store/index.js'
 import { subscribe } from '@nextcloud/event-bus'
-import { EDITOR_UPLOAD, HOOK_MENTION_SEARCH, HOOK_MENTION_INSERT, ATTACHMENT_RESOLVER } from './components/Editor.provider.js'
+import { EDITOR_UPLOAD, HOOK_MENTION_SEARCH, HOOK_MENTION_INSERT, ATTACHMENT_RESOLVER, OPEN_LINK_HANDLER } from './components/Editor.provider.js'
 import { ACTION_ATTACHMENT_PROMPT } from './components/Editor/MediaHandler.provider.js'
+import { openLink } from './helpers/links.js'
 // eslint-disable-next-line import/no-unresolved, n/no-missing-import
 import 'vite/modulepreload-polyfill'
 
@@ -184,6 +185,7 @@ window.OCA.Text.createEditor = async function({
 	onFileInsert = undefined,
 	onMentionSearch = undefined,
 	onMentionInsert = undefined,
+	openLinkHandler = undefined,
 	onSearch = undefined,
 }) {
 	const { default: MarkdownContentEditor } = await import(/* webpackChunkName: "editor" */'./components/Editor/MarkdownContentEditor.vue')
@@ -205,6 +207,9 @@ window.OCA.Text.createEditor = async function({
 				[EDITOR_UPLOAD]: !!sessionEditor,
 				[HOOK_MENTION_SEARCH]: sessionEditor ? true : onMentionSearch,
 				[HOOK_MENTION_INSERT]: sessionEditor ? true : onMentionInsert,
+				[OPEN_LINK_HANDLER]: {
+					openLink: openLinkHandler || openLink,
+				},
 				[ATTACHMENT_RESOLVER]: {
 					resolve(src, preferRaw) {
 						return [{

src/extensions/LinkBubble.js

@@ -21,7 +21,6 @@ const LinkBubble = Extension.create({
 		return [
 			linkBubble({
 				editor: this.editor,
-				parent: this.editor.contentComponent,
 			}),
 		]
 	},

src/helpers/links.js

@@ -56,8 +56,29 @@ const isLinkToSelfWithHash = function(href) {
 	return href?.startsWith('#') || href?.startsWith(locationNoHash + '#')
 }
 
+/**
+ * Open links, to be used as custom click handler
+ *
+ * @param {string} href the link href
+ */
+const openLink = function(href) {
+	const linkUrl = new URL(href, window.location.href)
+	// Consider rerouting links to Collectives if already inside Collectives app
+	const collectivesUrlBase = '/apps/collectives'
+	if (window.OCA.Collectives?.vueRouter
+		&& linkUrl.pathname.toString().startsWith(generateUrl(collectivesUrlBase))) {
+		const collectivesUrl = linkUrl.href.substring(
+			linkUrl.href.indexOf(collectivesUrlBase) + collectivesUrlBase.length,
+		)
+		window.OCA.Collectives.vueRouter.push(collectivesUrl)
+		return
+	}
+	window.open(linkUrl, '_blank')
+}
+
 export {
 	domHref,
-	parseHref,
 	isLinkToSelfWithHash,
+	openLink,
+	parseHref,
 }

src/plugins/LinkBubblePluginView.js

@@ -69,7 +69,7 @@ class LinkBubblePluginView {
 		}
 
 		this.#component ||= new VueRenderer(LinkBubbleView, {
-			parent: this.options.parent,
+			parent: this.options.editor.contentComponent,
 			propsData: {
 				editor: this.options.editor,
 				href: null,
@@ -109,7 +109,13 @@ class LinkBubblePluginView {
 	}
 
 	updateTooltip(view, { mark, nodeStart }) {
-		let referenceEl = view.nodeDOM(nodeStart)
+		let referenceEl
+		try {
+			referenceEl = view.nodeDOM(nodeStart)
+		} catch (e) {
+			// Prevent throwing error at rerouting in `openLink()`
+			return
+		}
 		if (Object.prototype.toString.call(referenceEl) === '[object Text]') {
 			referenceEl = referenceEl.parentElement
 		}