From 7c1dfd985067becbafa625cde20623f868568897 Mon Sep 17 00:00:00 2001
From: Carl Schwan <carl@carlschwan.eu>
Date: Thu, 13 Jan 2022 18:59:33 +0100
Subject: [PATCH] Improve layout for the public download page

This centers the content inside the pages, while also not hardcoded the
size of the footer.

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
---
 apps/files_sharing/templates/public.php |  2 +-
 core/css/public.scss                    | 31 +++++++++++++++++--------
 2 files changed, 22 insertions(+), 11 deletions(-)

diff --git a/apps/files_sharing/templates/public.php b/apps/files_sharing/templates/public.php
index 832e8a00b1a..33dd6ecd189 100644
--- a/apps/files_sharing/templates/public.php
+++ b/apps/files_sharing/templates/public.php
@@ -87,7 +87,7 @@ $maxUploadFilesize = min($upload_max_filesize, $post_max_size);
 				<?php if ($_['previewURL'] === $_['downloadURL'] && !$_['hideDownload']): ?>
 					<div class="directDownload">
 						<div>
-							<?php p($_['filename'])?> (<?php p($_['fileSize']) ?>)
+							<?php p($_['filename'])?>&nbsp;(<?php p($_['fileSize']) ?>)
 						</div>
 						<a href="<?php p($_['downloadURL']); ?>" id="downloadFile" class="button">
 							<span class="icon icon-download"></span>
diff --git a/core/css/public.scss b/core/css/public.scss
index b171dd4970d..31e48f3c7cc 100644
--- a/core/css/public.scss
+++ b/core/css/public.scss
@@ -1,8 +1,25 @@
-$footer-height: 65px;
-
 #body-public {
-	.header-right {
+	min-height: 100vh;
+	display: flex;
+	flex-direction: column;
 
+	#content {
+		flex-grow: 2;
+		min-height: initial;
+
+		/** Center the shared content inside the page */
+		&.app-files_sharing {
+			justify-content: center;
+			align-items: center;
+			#app-content {
+				min-height: inherit;
+				padding-left: 1rem;
+				padding-right: 1rem;
+			}
+		}
+	}
+
+	.header-right {
 		#header-primary-action a {
 			color: var(--color-primary-text);
 		}
@@ -35,12 +52,6 @@ $footer-height: 65px;
 		}
 	}
 
-	#content {
-		// 100% - footer
-		min-height: calc(100% - #{$footer-height});
-
-	}
-
 	/** don't apply content header padding on the base layout */
 	&.layout-base #content {
 		padding-top: 0;
@@ -72,8 +83,8 @@ $footer-height: 65px;
 		display: flex;
 		align-items: center;
 		justify-content: center;
-		height: $footer-height;
 		flex-direction: column;
+		padding: 0.5rem;
 		p {
 			text-align: center;
 			color: var(--color-text-lighter);