Fix footer not hidden when Talk sidebar is shown

When the Talk sidebar is shown the footer in public pages is moved into the "#app-content" element, so this needs to be taken into account when hiding the footer element. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
nextcloud · Jun 27, 2022 · e01afb8 · e01afb8
1 parent 40ebf9b
commit e01afb8
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
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