fix(LinkBubble): Treat links to hashes in same page directly

Fixes: #5495 Signed-off-by: Jonas <jonas@freesources.org>
nextcloud · Mar 18, 2024 · 7d137e6 · 7d137e6
1 parent 08ccd25
commit 7d137e6
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 5 deletions.
diff --git a/src/extensions/LinkBubblePluginView.js b/src/extensions/LinkBubblePluginView.js
@@ -1,7 +1,7 @@
 import { VueRenderer } from '@tiptap/vue-2'
 import tippy from 'tippy.js'
 import debounce from 'debounce'
-import { domHref } from '../helpers/links.js'
+import { domHref, isLinkToSelfWithHash } from '../helpers/links.js'
 import LinkBubbleView from '../components/Link/LinkBubbleView.vue'
 
 import { getViewerVue } from '../ViewerVue.js'
@@ -220,7 +220,7 @@ class LinkBubblePluginView {
 		}
 
 		// Don't open link bubble for anchor links
-		if (linkMark.attrs.href.startsWith('#')) {
+		if (isLinkToSelfWithHash(linkMark.attrs.href)) {
 			return false
 		}
 

diff --git a/src/helpers/links.js b/src/helpers/links.js
@@ -68,7 +68,13 @@ const parseHref = function(dom) {
 	return ref
 }
 
+const isLinkToSelfWithHash = function(href) {
+	const locationNoHash = window.location.origin + window.location.pathname + window.location.search
+	return href?.startsWith('#') || href?.startsWith(locationNoHash + '#')
+}
+
 export {
 	domHref,
 	parseHref,
+	isLinkToSelfWithHash,
 }
diff --git a/src/nodes/Preview.js b/src/nodes/Preview.js
@@ -21,7 +21,7 @@
  */
 
 import { Node, isNodeActive, getNodeType } from '@tiptap/core'
-import { domHref, parseHref } from './../helpers/links.js'
+import { domHref, parseHref, isLinkToSelfWithHash } from './../helpers/links.js'
 import { VueNodeViewRenderer } from '@tiptap/vue-2'
 
 import Preview from './Preview.vue'
@@ -149,7 +149,7 @@ function previewPossible({ selection }) {
 		return false
 	}
 	const href = extractHref($from.nodeAfter)
-	if (!href || href.startsWith('#')) {
+	if (!href || isLinkToSelfWithHash('#')) {
 		return false
 	}
 	return true

diff --git a/src/plugins/links.js b/src/plugins/links.js
@@ -21,6 +21,7 @@
  */
 
 import { Plugin, PluginKey } from '@tiptap/pm/state'
+import { isLinkToSelfWithHash } from '../helpers/links.js'
 
 export function linkClicking() {
 	return new Plugin({
@@ -53,7 +54,8 @@ export function linkClicking() {
 					const linkEl = event.target.closest('a')
 					if (event.button === 0 && linkEl) {
 						event.preventDefault()
-						if (linkEl.attributes.href?.value?.startsWith('#')) {
+						const href = linkEl.attributes.href?.value
+						if (isLinkToSelfWithHash(linkEl.attributes.href?.value)) {
 							// Open anchor links directly
 							location.href = linkEl.attributes.href.value
 						} else if (event.ctrlKey || event.metaKey) {