From a4db5c9bd264d0d8e2a65814fa5d4131de70ec24 Mon Sep 17 00:00:00 2001 From: windingwind <33902321+windingwind@users.noreply.github.com> Date: Wed, 9 Apr 2025 06:34:05 +0200 Subject: [PATCH] Fix can't move second-to-last item pane section down (#5187) --- chrome/content/zotero/elements/itemPaneSidenav.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/chrome/content/zotero/elements/itemPaneSidenav.js b/chrome/content/zotero/elements/itemPaneSidenav.js index f960a47297..6b4195a7e1 100644 --- a/chrome/content/zotero/elements/itemPaneSidenav.js +++ b/chrome/content/zotero/elements/itemPaneSidenav.js @@ -767,10 +767,14 @@ return; } let targetWrapper = enabledWrappers[targetIndex]; + let actualIndex; if (targetWrapper) { - // Insert at the index of the previous wrapper - this.changePaneOrder(paneID, this._wrappers.indexOf(targetWrapper)); + actualIndex = this._wrappers.indexOf(targetWrapper); + } else { + actualIndex = this._wrappers.length; } + // Insert at the index of the previous wrapper + this.changePaneOrder(paneID, actualIndex); } handleButtonDragStart = (event) => {