From 39fc1c72fc2081813a1d261817815e415c9f2424 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Thu, 14 Jun 2018 16:09:22 +0200 Subject: [PATCH] Fix regression causing the pages to be offset horizontally in Presentation Mode (PR 8993 follow-up) This is a regression from PR 8993; it causes the pages to be offset horizontally in Presentation Mode, if and only if the sidebar is currently open when the user triggers Presentation Mode. Please note that while this doesn't seem to affect Firefox, both Chrome and IE are however affected. Interestingly enough, despite the Chrome extension being affected as well, I cannot find any issue filed about this. (Either Presentation Mode isn't used much at all, or users don't open the sidebar before entering Presentation Mode.) --- web/viewer.css | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/web/viewer.css b/web/viewer.css index 9c3a3cada..4115fe93d 100644 --- a/web/viewer.css +++ b/web/viewer.css @@ -175,7 +175,8 @@ html[dir='rtl'] #sidebarContent { bottom: 0; left: 0; outline: none; - +} +#viewerContainer:not(.pdfPresentationMode) { transition-duration: 200ms; transition-timing-function: ease; } @@ -191,12 +192,12 @@ html[dir='rtl'] #viewerContainer { transition-duration: 0s; } -html[dir='ltr'] #outerContainer.sidebarOpen #viewerContainer { +html[dir='ltr'] #outerContainer.sidebarOpen #viewerContainer:not(.pdfPresentationMode) { transition-property: left; left: 200px; left: var(--sidebar-width); } -html[dir='rtl'] #outerContainer.sidebarOpen #viewerContainer { +html[dir='rtl'] #outerContainer.sidebarOpen #viewerContainer:not(.pdfPresentationMode) { transition-property: right; right: 200px; right: var(--sidebar-width);