From 946a1a959f563c3ce0efc4d01aba446182c9e421 Mon Sep 17 00:00:00 2001
From: Ryan Mathis <rmathis@unicon.net>
Date: Wed, 13 Nov 2019 15:05:33 -0700
Subject: [PATCH] SHIBUI-1458 Fixed issue with label on data list components

---
 .../app/schema-form/widget/datalist/datalist.component.html   | 4 ++--
 ui/src/app/schema-form/widget/datalist/datalist.component.ts  | 4 ++++
 ui/src/app/schema-form/widget/select/select.component.html    | 3 +--
 3 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/ui/src/app/schema-form/widget/datalist/datalist.component.html b/ui/src/app/schema-form/widget/datalist/datalist.component.html
index 58cf01c46..ae5351a9b 100644
--- a/ui/src/app/schema-form/widget/datalist/datalist.component.html
+++ b/ui/src/app/schema-form/widget/datalist/datalist.component.html
@@ -1,7 +1,7 @@
 <div class="widget form-group">
     <label [for]="formProperty._canonicalPath" class="d-flex justify-content-between control-label">
         <span [ngClass]="{'sr-only': formProperty.parent.schema.type === 'array'}">
-            <translate-i18n [key]="schema.title">{{ schema.title }}</translate-i18n>
+            <translate-i18n [key]="title">{{ title }}</translate-i18n>
             <i class="fa fa-asterisk text-danger" aria-hidden="true" *ngIf="required"></i>
         </span>
         <span *ngIf="schema.description">
@@ -19,7 +19,7 @@
         [attr.placeholder]="schema.placeholder || '' | translate"
         [attr.disabled]="schema.readOnly?true:null"
         role="textbox"
-        [attr.aria-label]="schema.title | translate">
+        [attr.aria-label]="title | translate">
     </auto-complete>
     <small class="form-text text-danger" *ngIf="errorMessages && errorMessages.length && control.touched">
         <ng-container *ngFor="let error of errorMessages; let first=first;">
diff --git a/ui/src/app/schema-form/widget/datalist/datalist.component.ts b/ui/src/app/schema-form/widget/datalist/datalist.component.ts
index aecbe2fc3..08d9c5b48 100644
--- a/ui/src/app/schema-form/widget/datalist/datalist.component.ts
+++ b/ui/src/app/schema-form/widget/datalist/datalist.component.ts
@@ -28,6 +28,10 @@ export class DatalistComponent extends ControlWidget implements AfterViewInit {
         return this.widgetService.isRequired(this.formProperty);
     }
 
+    get title(): string {
+        return this.schema.title || this.formProperty.parent.schema.title;
+    }
+
     getError(error: string): string {
         return HARD_CODED_REQUIRED_MSG.test(error) ? 'message.required' : error;
     }
diff --git a/ui/src/app/schema-form/widget/select/select.component.html b/ui/src/app/schema-form/widget/select/select.component.html
index f3f709717..832c9bf3f 100644
--- a/ui/src/app/schema-form/widget/select/select.component.html
+++ b/ui/src/app/schema-form/widget/select/select.component.html
@@ -19,8 +19,7 @@
         [formControl]="control"
         [attr.name]="name"
         class="form-control"
-        [attr.aria-label]="schema.title"
-        [ngClass]="{'text-muted': !this.formProperty.value}">
+        [attr.aria-label]="schema.title">
         <option selected value="">
             <translate-i18n *ngIf="schema.placeholder" [key]="schema.placeholder">{{ schema.placeholder }}</translate-i18n>
             <translate-i18n *ngIf="!schema.placeholder" [key]="schema.title">{{ schema.title }}</translate-i18n>