Merge pull request #1241 from nextcloud/fix/hide-empty-face-title

nextcloud · Sep 14, 2022 · ac8c298 · ac8c298
2 parents 7cad5e8 + 0d42e71
commit ac8c298
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 8 deletions.
diff --git a/js/photos-main.js b/js/photos-main.js
diff --git a/js/photos-main.js.map b/js/photos-main.js.map
diff --git a/js/photos-src_views_Faces_vue.js b/js/photos-src_views_Faces_vue.js
diff --git a/js/photos-src_views_Faces_vue.js.map b/js/photos-src_views_Faces_vue.js.map
diff --git a/src/components/FaceCover.vue b/src/components/FaceCover.vue
@@ -31,8 +31,8 @@
 				:style="coverDimensions">
 		</div>
 		<div class="face-cover__details">
-			<div class="face-cover__details__first-line">
-				<h2 :class="{'face-cover__details__name': true, 'hidden-visually': baseName.match(/^[0-9]+$/)}">
+			<div v-if="!baseName.match(/^[0-9]+$/)" class="face-cover__details__first-line">
+				<h2 class="face-cover__details__name">
 					{{ baseName }}
 				</h2>
 			</div>
@@ -157,6 +157,7 @@ export default {
 		}
 
 		&__second-line {
+			margin-top: 6px;
 			color: var(--color-text-maxcontrast);
 		}