tag multiselect shouldnt use model updating on blur

griffithlab · Jan 31, 2024 · 071a8a2 · 071a8a2
1 parent 3d72dae
commit 071a8a2
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/client/src/app/forms/types/tag-input/tag-input.module.ts b/client/src/app/forms/types/tag-input/tag-input.module.ts
@@ -16,6 +16,13 @@ const typeConfig: ConfigOption = {
       name: 'tag-input',
       wrappers: ['form-field'],
       component: CvcTagInputField,
+      defaultOptions: {
+        modelOptions: {
+          // update model when focus leaves field
+          // (template's keydown.enter listener blurs the field, updating the model)
+          updateOn: 'blur',
+        },
+      },
     },
     {
       // for use in repeat-field types

diff --git a/client/src/app/forms/types/tag-input/tag-input.type.ts b/client/src/app/forms/types/tag-input/tag-input.type.ts
@@ -36,11 +36,6 @@ const TagInputMixin = mixin(
 })
 export class CvcTagInputField extends TagInputMixin implements AfterViewInit {
   defaultOptions: Partial<FieldTypeConfig<CvcTagInputProps>> = {
-    modelOptions: {
-      // update model when focus leaves field
-      // (template's keydown.enter listener blurs the field, updating the model)
-      updateOn: 'blur',
-    },
     props: {
       label: 'Enter value',
     },