Merge pull request #615 from nextcloud/fix-footer-not-hidden-when-tal…

…k-sidebar-is-shown
nextcloud · Jun 28, 2022 · 4f95b24 · 4f95b24
2 parents 4daeddd + e01afb8
commit 4f95b24
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 19 deletions.
diff --git a/css/style.css b/css/style.css
diff --git a/js/files_pdfviewer-public.js b/js/files_pdfviewer-public.js
diff --git a/js/files_pdfviewer-public.js.map b/js/files_pdfviewer-public.js.map
diff --git a/src/public.js b/src/public.js
@@ -45,7 +45,10 @@ window.addEventListener('DOMContentLoaded', function() {
 		const page = location.hash.split('page=')[1] || 0
 		const contentElmt = document.getElementById('files-public-content')
 		const sharingTokenElmt = document.getElementById('sharingToken')
-		const footerElmt = document.querySelector('body > footer')
+		// By default the footer is a direct child of the body, but if the Talk
+		// sidebar is loaded it is moved into the app content. In all cases the
+		// footer is hidden to give the PDF viewer the full height.
+		const footerElmt = document.querySelector('body > footer') || document.querySelector('#app-content > footer')
 		const mainContent = document.querySelector('#content')
 
 		const sharingToken = sharingTokenElmt.value