From 817ca999e3d29d297cde7dcbfafa40783e88b7a5 Mon Sep 17 00:00:00 2001 From: Dan Stillman Date: Mon, 3 Oct 2022 05:10:39 -0400 Subject: [PATCH] Swap "Split Vertically" and "Split Horizontally" in PDF reader --- chrome/content/zotero/reader.xhtml | 12 ++++++------ chrome/content/zotero/xpcom/reader.js | 14 +++++++------- chrome/content/zotero/zoteroPane.xhtml | 14 +++++++------- 3 files changed, 20 insertions(+), 20 deletions(-) diff --git a/chrome/content/zotero/reader.xhtml b/chrome/content/zotero/reader.xhtml index 58b112d9a8..01a5d035c9 100644 --- a/chrome/content/zotero/reader.xhtml +++ b/chrome/content/zotero/reader.xhtml @@ -221,18 +221,18 @@ oncommand="menuCmd('zoomPageHeight')" /> - + this._splitVertically()); - popup.appendChild(menuitem); // Split Horizontally menuitem = this._window.document.createXULElement('menuitem'); menuitem.setAttribute('label', Zotero.getString('pdfReader.splitHorizontally')); @@ -716,6 +709,13 @@ class ReaderInstance { menuitem.setAttribute('checked', this.isSplitHorizontallyActive()); menuitem.addEventListener('command', () => this._splitHorizontally()); popup.appendChild(menuitem); + // Split Vertically + menuitem = this._window.document.createXULElement('menuitem'); + menuitem.setAttribute('label', Zotero.getString('pdfReader.splitVertically')); + menuitem.setAttribute('type', 'checkbox'); + menuitem.setAttribute('checked', this.isSplitVerticallyActive()); + menuitem.addEventListener('command', () => this._splitVertically()); + popup.appendChild(menuitem); // Separator popup.appendChild(this._window.document.createXULElement('menuseparator')); // Next page diff --git a/chrome/content/zotero/zoteroPane.xhtml b/chrome/content/zotero/zoteroPane.xhtml index 3a52e3e42a..8450bcdb3a 100644 --- a/chrome/content/zotero/zoteroPane.xhtml +++ b/chrome/content/zotero/zoteroPane.xhtml @@ -393,13 +393,6 @@ oncommand="ZoteroStandalone.onReaderCmd('zoomPageHeight')" /> - +