DatePicker: improve form change validation (#12348)

watch.value will keep firing during time picker selection it now detects whether panel is closed before firing el.form.change
ElemeFE · Aug 15, 2018 · b70da53 · b70da53
1 parent 02176e2
commit b70da53
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/packages/date-picker/src/picker.vue b/packages/date-picker/src/picker.vue
@@ -429,7 +429,7 @@ export default {
       }
     },
     value(val, oldVal) {
-      if (!valueEquals(val, oldVal)) {
+      if (!valueEquals(val, oldVal) && !this.pickerVisible) {
         this.dispatch('ElFormItem', 'el.form.change', val);
       }
     }
@@ -893,6 +893,7 @@ export default {
       // determine user real change only
       if (!valueEquals(val, this.valueOnOpen)) {
         this.$emit('change', val);
+        this.dispatch('ElFormItem', 'el.form.change', val);
         this.valueOnOpen = val;
       }
     },