From 0b5012555d94532083c76d14803134d276da4cfa Mon Sep 17 00:00:00 2001 From: Martynas Bagdonas Date: Thu, 19 Jun 2025 16:54:34 +0300 Subject: [PATCH] Fix regression triggering multiple reader context menus Fixes https://forums.zotero.org/discussion/124939/persistent-context-menu-in-the-snapshot-reader Fixes https://forums.zotero.org/discussion/124938/double-context-menus-on-annotations-in-the-pdf-reader --- chrome/content/zotero/xpcom/reader.js | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/chrome/content/zotero/xpcom/reader.js b/chrome/content/zotero/xpcom/reader.js index 48090c5041..1b5227ce81 100644 --- a/chrome/content/zotero/xpcom/reader.js +++ b/chrome/content/zotero/xpcom/reader.js @@ -1191,6 +1191,14 @@ class ReaderInstance { _handleReaderTextboxContextMenuOpen = (event) => { this._window.goUpdateGlobalEditMenuItems(true); + if ( + event.view !== this._iframeWindow + // For text annotation box + && (this._type !== 'pdf' || event.target?.nodeName !== 'TEXTAREA') + ) { + return; + } + let iframeWindow = event.target.ownerGlobal; this._window.MozXULElement.insertFTLIfNeeded("toolkit/global/textActions.ftl");