Merge pull request #11511 from nextcloud/stable14-share-header

[stable14] Fix share header text on small widths
nextcloud · Oct 1, 2018 · e7cdacf · e7cdacf
2 parents 7841c95 + 687d050
commit e7cdacf
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/apps/files_sharing/css/public.scss b/apps/files_sharing/css/public.scss
@@ -165,6 +165,8 @@ thead {
 	font-weight: 300;
 	font-size: 11px;
 	opacity: .57;
+	overflow: hidden;
+	text-overflow: ellipsis;
 }
 
 #note-content {
@@ -186,4 +188,4 @@ thead {
 			}
 		}
 	}
-}
+}
diff --git a/core/css/header.scss b/core/css/header.scss
@@ -65,6 +65,8 @@
 	display: flex;
 	align-items: center;
 	flex-wrap: wrap;
+	overflow: hidden;
+
 	&:focus {
 		opacity: .75;
 	}
@@ -151,6 +153,7 @@
 		flex: 0 0;
 		flex-grow: 1;
 		white-space: nowrap;
+		overflow: hidden;
 	}
 
 	#header-right, .header-right {
@@ -206,14 +209,16 @@
 	}
 }
 
-/* show appname next to logo */
+/* only used for public share pages now as we have the app icons when logged in */
 .header-appname {
 	color: var(--color-primary-text);
 	font-size: 16px;
 	font-weight: 300;
 	margin: 0;
 	padding: 0;
 	padding-right: 5px;
+	overflow: hidden;
+	text-overflow: ellipsis;
 }
 
 /* do not show menu toggle on public share links as there is no menu */

diff --git a/core/css/public.scss b/core/css/public.scss
@@ -8,8 +8,6 @@ $footer-height: 65px;
 		}
 
 		#header-secondary-action {
-			margin-right: 13px;
-
 			ul li {
 				min-width: 270px;
 			}