Update how navigation sidebar visibility is controlled (#1221)

2 files changed, 6 insertions, 9 deletions

diff --git a/ext/mixed/css/display.css b/ext/mixed/css/display.css
index 6bea1ad2..a7c3e368 100644
--- a/ext/mixed/css/display.css
+++ b/ext/mixed/css/display.css
@@ -382,13 +382,10 @@ a {
     background-color: var(--sidebar-background-color);
     z-index: 10;
     position: relative;
-    display: block;
-}
-.content-sidebar[hidden] {
     display: none;
 }
-.content-sidebar[hidden][data-has-navigation-previous=true],
-.content-sidebar[hidden][data-has-navigation-next=true],
+:root[data-has-navigation-previous=true] .content-sidebar,
+:root[data-has-navigation-next=true] .content-sidebar,
 :root[data-popup-action-bar-visibility=always] .content-sidebar,
 :root[data-profile-panel-visible=true] .content-sidebar {
     display: block;
diff --git a/ext/mixed/js/display.js b/ext/mixed/js/display.js
index 2b3ea21c..b43b4dca 100644
--- a/ext/mixed/js/display.js
+++ b/ext/mixed/js/display.js
@@ -97,7 +97,6 @@ class Display extends EventDispatcher {
         this._contentScrollElement = document.querySelector('#content-scroll');
         this._contentScrollBodyElement = document.querySelector('#content-body');
         this._windowScroll = new WindowScroll(this._contentScrollElement);
-        this._contentSidebar = document.querySelector('#content-sidebar');
         this._closeButton = document.querySelector('#close-button');
         this._navigationPreviousButton = document.querySelector('#navigate-previous-button');
         this._navigationNextButton = document.querySelector('#navigate-next-button');
@@ -1074,9 +1073,10 @@ class Display extends EventDispatcher {
     }
 
     _updateNavigation(previous, next) {
-        if (this._contentSidebar !== null) {
-            this._contentSidebar.dataset.hasNavigationPrevious = `${previous}`;
-            this._contentSidebar.dataset.hasNavigationNext = `${next}`;
+        const {documentElement} = document;
+        if (documentElement !== null) {
+            documentElement.dataset.hasNavigationPrevious = `${previous}`;
+            documentElement.dataset.hasNavigationNext = `${next}`;
         }
         if (this._navigationPreviousButton !== null) {
             this._navigationPreviousButton.disabled = !previous;