Implement link copying option for PDF view context menu

Fixes zotero/zotero#3494
zotero · Oct 14, 2024 · 85934b1 · 85934b1
1 parent ffc5593
commit 85934b1
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 1 deletion.
diff --git a/src/common/context-menu.js b/src/common/context-menu.js
@@ -91,6 +91,15 @@ export function createViewContextMenu(reader, params) {
 		x: params.x,
 		y: params.y,
 		itemGroups: createItemGroup([
+			[
+				{
+					label: reader._getString('general.copyLink'),
+					disabled: !(params.overlay && params.overlay.type === 'external-link' && !reader.canCopy),
+					onCommand: () => {
+						navigator.clipboard.writeText(params.overlay.url);
+					}
+				}
+			],
 			[
 				{
 					label: reader._getString('general.copy'),

diff --git a/src/en-us.strings.js b/src/en-us.strings.js
@@ -42,6 +42,7 @@ export default {
 	'general.continue': 'Continue',
 	'general.dontShowAgain': 'Don’t Show Again',
 	'general.copy': 'Copy',
+	'general.copyLink': 'Copy Link',
 	'general.keys.ctrlShift': 'Ctrl+Shift+',
 	'general.close': 'Close',
 	'general.create': 'Create',

diff --git a/src/pdf/pdf-view.js b/src/pdf/pdf-view.js
@@ -1680,7 +1680,11 @@ class PDFView {
 				if (this._selectedAnnotationIDs.length !== 0) {
 					this._onSelectAnnotations([], event);
 				}
-				this._onOpenViewContextMenu({ x: br.x + event.clientX, y: br.y + event.clientY });
+				let overlay;
+				if (position) {
+					overlay = this._getSelectableOverlay(position);
+				}
+				this._onOpenViewContextMenu({ x: br.x + event.clientX, y: br.y + event.clientY, overlay });
 			}
 			else if (!selectedAnnotations.includes(selectableAnnotation)) {
 				this._onSelectAnnotations([selectableAnnotation.id], event);