From 64489b243ffd217d97c64f52c67674eb2c6e01a2 Mon Sep 17 00:00:00 2001 From: windingwind <33902321+windingwind@users.noreply.github.com> Date: Sun, 3 Mar 2024 18:17:31 +0800 Subject: [PATCH] fx115: fix context pane sidenav render --- chrome/content/zotero/contextPane.js | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/chrome/content/zotero/contextPane.js b/chrome/content/zotero/contextPane.js index 336496c88c..65297d690b 100644 --- a/chrome/content/zotero/contextPane.js +++ b/chrome/content/zotero/contextPane.js @@ -136,8 +136,11 @@ var ZoteroContextPane = new function () { _contextPaneSplitter.setAttribute('hidden', true); _contextPane.classList.add('stacked'); _contextPane.classList.remove('standard'); - _sidenav.classList.add('stacked'); - _contextPaneInner.append(_sidenav); + _readerSidenav.classList.add('stacked'); + if (_readerSidenav.parentElement != _contextPaneInner) { + _contextPaneInner.append(_readerSidenav); + _readerSidenav.render(); + } // Fx115: in stacked layout, make contextPane occupy all width and remove min-height // needed for standard layout _contextPane.style.width = 'auto'; @@ -149,8 +152,11 @@ var ZoteroContextPane = new function () { _contextPaneSplitterStacked.setAttribute('state', 'open'); _contextPane.classList.add('standard'); _contextPane.classList.remove('stacked'); - _sidenav.classList.remove('stacked'); - _contextPane.append(_sidenav); + _readerSidenav.classList.remove('stacked'); + if (_readerSidenav.parentElement != _contextPane) { + _contextPane.append(_readerSidenav); + _readerSidenav.render(); + } // FX115: in standard layout, make contextPane have the width it's supposed to and // force it to occupy all height available _contextPaneInner.style.minHeight = `100%`;