Merge pull request #50602 from nextcloud/fix/template-field-title

fix: template field title
nextcloud · Feb 13, 2025 · 6e1d9a2 · 6e1d9a2
2 parents 4905255 + 729e278
commit 6e1d9a2
Show file tree

Hide file tree

Showing 12 changed files with 20 additions and 12 deletions.
diff --git a/apps/files/src/components/TemplateFiller.vue b/apps/files/src/components/TemplateFiller.vue
@@ -4,13 +4,18 @@
 -->
 
 <template>
-	<NcModal>
+	<NcModal label-id="template-field-modal__label">
 		<div class="template-field-modal__content">
 			<form>
-				<h3>{{ t('files', 'Fill template fields') }}</h3>
+				<h3 id="template-field-modal__label">
+					{{ t('files', 'Fill template fields') }}
+				</h3>
 
 				<div v-for="field in fields" :key="field.index">
-					<component :is="getFieldComponent(field.type)" :field="field" @input="trackInput" />
+					<component :is="getFieldComponent(field.type)"
+						v-if="fieldHasLabel(field)"
+						:field="field"
+						@input="trackInput" />
 				</div>
 			</form>
 		</div>
@@ -80,6 +85,9 @@ export default defineComponent({
 
 			return `Template${fieldComponentType}Field`
 		},
+		fieldHasLabel(field) {
+			return field.name || field.alias
+		},
 		async submit() {
 			this.loading = true
 

diff --git a/apps/files/src/components/TemplateFiller/TemplateCheckboxField.vue b/apps/files/src/components/TemplateFiller/TemplateCheckboxField.vue
@@ -40,7 +40,7 @@ export default defineComponent({
 
 	computed: {
 		fieldLabel() {
-			const label = this.field.name ?? this.field.alias ?? 'Unknown field'
+			const label = this.field.name || this.field.alias
 
 			return label.charAt(0).toUpperCase() + label.slice(1)
 		},

diff --git a/apps/files/src/components/TemplateFiller/TemplateRichTextField.vue b/apps/files/src/components/TemplateFiller/TemplateRichTextField.vue
@@ -45,7 +45,7 @@ export default defineComponent({
 
 	computed: {
 		fieldLabel() {
-			const label = this.field.name ?? this.field.alias ?? 'Unknown field'
+			const label = this.field.name || this.field.alias
 
 			return (label.charAt(0).toUpperCase() + label.slice(1))
 		},

diff --git a/dist/4341-4341.js b/dist/4341-4341.js
diff --git a/dist/465-465.js.license → dist/4341-4341.js.license b/dist/465-465.js.license → dist/4341-4341.js.license
diff --git a/dist/4341-4341.js.map b/dist/4341-4341.js.map
diff --git a/dist/4341-4341.js.map.license b/dist/4341-4341.js.map.license
@@ -0,0 +1 @@
+4341-4341.js.license
diff --git a/dist/465-465.js b/dist/465-465.js
diff --git a/dist/465-465.js.map b/dist/465-465.js.map
diff --git a/dist/465-465.js.map.license b/dist/465-465.js.map.license
diff --git a/dist/files-init.js b/dist/files-init.js
diff --git a/dist/files-init.js.map b/dist/files-init.js.map