Fix public layout header title & description

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

diff --git a/apps/files_sharing/css/public.scss b/apps/files_sharing/css/public.scss
index a8207167c2c7d681afb24082fc76f2f3c2504171..fb50dd2f25331aa0c612021400bda20f2648da13 100644 (file)
--- a/apps/files_sharing/css/public.scss
+++ b/apps/files_sharing/css/public.scss
@@ -168,22 +168,6 @@ thead {
        text-align: left;
 }
 
-/* Needed to ellipsize long header text on share page */
-#body-login #header-left,
-#body-login .header-left {
-       overflow: hidden;
-}
-
-#header .header-shared-by  {
-       color: var(--color-primary-text);
-       position: relative;
-       font-weight: 300;
-       font-size: 11px;
-       line-height: 11px;
-       overflow: hidden;
-       text-overflow: ellipsis;
-}
-
 #note-content {
        padding: 5px;
        display:inline-block;

diff --git a/core/css/header.scss b/core/css/header.scss
index 2b63dc978ace6e414636f65a0cd77367e207e1bf..ace4f898bcedad51112aa961bc1a9df51626f140 100644 (file)
--- a/core/css/header.scss
+++ b/core/css/header.scss
@@ -63,6 +63,7 @@
        box-sizing: border-box;
        opacity: 1;
        align-items: center;
+       display: flex;
        flex-wrap: wrap;
        overflow: hidden;
 
@@ -198,9 +199,9 @@
        }
 
        #header-left, .header-left {
-               flex: 0 0;
-               flex-grow: 1;
+               flex: 1 0;
                white-space: nowrap;
+               min-width: 0;
        }
 
        #header-right, .header-right {
@@ -256,6 +257,7 @@
        }
 }
 
+/* TODO: move into minimal css file for public shared template */
 /* only used for public share pages now as we have the app icons when logged in */
 .header-appname {
        color: var(--color-primary-text);
@@ -266,6 +268,18 @@
        padding-right: 5px;
        overflow: hidden;
        text-overflow: ellipsis;
+       // Take full width to push the header-shared-by bellow (if any)
+       flex: 1 1 100%;
+}
+
+.header-shared-by  {
+       color: var(--color-primary-text);
+       position: relative;
+       font-weight: 300;
+       font-size: 11px;
+       line-height: 11px;
+       overflow: hidden;
+       text-overflow: ellipsis;
 }
 
 /* do not show menu toggle on public share links as there is no menu */

diff --git a/core/templates/layout.public.php b/core/templates/layout.public.php
index bbfa264d17172e0d52998f366da38596d60c2a48..f041be7f26da2404a55ca113afa1cb8b72a28cb2 100644 (file)
--- a/core/templates/layout.public.php
+++ b/core/templates/layout.public.php
@@ -39,11 +39,17 @@
                        <span id="nextcloud">
                                <div class="logo logo-icon svg"></div>
                                <h1 class="header-appname">
-                                       <?php if (isset($template)) { p($template->getHeaderTitle()); } else { p($theme->getName());} ?>
+                                       <?php if (isset($template) && $template->getHeaderTitle() !== '') {
+                                               p($template->getHeaderTitle());
+                                       } else {
+                                               p($theme->getName());
+                                       } ?>
                                </h1>
+                               <?php if (isset($template) && $template->getHeaderDetails() !== '') { ?>
                                <div class="header-shared-by">
-                                       <?php if (isset($template)) { p($template->getHeaderDetails()); } ?>
+                                       <?php p($template->getHeaderDetails()); ?>
                                </div>
+                               <?php } ?>
                        </span>
                </div>