fix(form): ensure that the hidden fields of the form are verified pro…

…perly, fix #413
vbenjs · Mar 25, 2021 · 237f41d · 237f41d
1 parent 5c57a1d
commit 237f41d
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 3 deletions.
diff --git a/CHANGELOG.zh_CN.md b/CHANGELOG.zh_CN.md
@@ -20,6 +20,7 @@
 - 确保 progress 进度条正确关闭
 - 修复表格勾选列配置失效问题
 - 确保一级菜单可以被隐藏
+- 确保表单隐藏字段校验正常
 
 ### 🎫 Chores
 

diff --git a/src/components/Form/src/components/FormItem.vue b/src/components/Form/src/components/FormItem.vue
@@ -156,9 +156,9 @@
           if (!isShow) {
             rule.required = false;
           }
-          if (rule.required && component) {
+          if (component) {
             if (!Reflect.has(rule, 'type')) {
-              rule.type = 'string';
+              rule.type = component === 'InputNumber' ? 'number' : 'string';
             }
             const joinLabel = Reflect.has(props.schema, 'rulesMessageJoinLabel')
               ? rulesMessageJoinLabel

diff --git a/src/design/ant/index.less b/src/design/ant/index.less
@@ -14,7 +14,7 @@
 }
 
 span.anticon:not(.app-iconify) {
-  vertical-align: 0.135em;
+  vertical-align: 0.125em !important;
 }
 
 .ant-back-top {

diff --git a/src/views/demo/form/RuleForm.vue b/src/views/demo/form/RuleForm.vue
@@ -38,6 +38,14 @@
       },
       required: true,
     },
+    {
+      field: 'id',
+      label: 'id',
+      required: true,
+      defaultValue: 0,
+      component: 'InputNumber',
+      show: false,
+    },
     {
       field: 'field3',
       component: 'DatePicker',