From 57916c032f176041f72055b7761a312e96c5d12c Mon Sep 17 00:00:00 2001
From: Abe Jellinek <jellinek@berkeley.edu>
Date: Thu, 27 Oct 2022 16:54:50 -0400
Subject: [PATCH] Attachment box: Add ability to copy URL of snapshot/link
 (#2748)

Fixes #2747
---
 .../content/zotero/elements/attachmentBox.js  | 20 ++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/chrome/content/zotero/elements/attachmentBox.js b/chrome/content/zotero/elements/attachmentBox.js
index e9d43566e3..e87e106ca7 100644
--- a/chrome/content/zotero/elements/attachmentBox.js
+++ b/chrome/content/zotero/elements/attachmentBox.js
@@ -81,6 +81,12 @@
 					<note-editor id="attachment-note-editor" notitle="1" flex="1"/>
 					
 					<button id="select-button" hidden="true"/>
+					
+					<popupset>
+						<menupopup id="url-menu">
+							<menuitem id="url-menuitem-copy"/>
+						</menupopup>
+					</popupset>
 				</vbox>
 			`, ['chrome://zotero/locale/zotero.dtd']);
 		}
@@ -194,6 +200,16 @@
 				}
 			});
 
+			this._id('url').addEventListener('contextmenu', (event) => {
+				this._id('url-menu').openPopupAtScreen(event.screenX, event.screenY, true);
+			});
+
+			let copyMenuitem = this._id('url-menuitem-copy');
+			copyMenuitem.label = Zotero.getString('general.copy');
+			copyMenuitem.addEventListener('command', () => {
+				Zotero.Utilities.Internal.copyTextToClipboard(this.item.getField('url'));
+			});
+
 			this._notifierID = Zotero.Notifier.registerObserver(this, ['item'], 'attachmentbox');
 		}
 
@@ -279,7 +295,9 @@
 					urlField.setAttribute('hidden', false);
 					if (this.clickableLink) {
 						urlField.onclick = function (event) {
-							ZoteroPane_Local.loadURI(this.value, event);
+							if (event.button == 0) {
+								ZoteroPane_Local.loadURI(this.value, event);
+							}
 						};
 						urlField.className = 'zotero-text-link';
 					}