From fc2a15c75e273f8b40647784001f24e313fb1449 Mon Sep 17 00:00:00 2001
From: Justineo <justice360@gmail.com>
Date: Tue, 12 Jun 2018 23:54:36 +0800
Subject: [PATCH] feat: add role for Fieldset, use more generic wrapper for
 Field

---
 packages/veui/src/components/Form/Field.vue    | 10 +++++-----
 packages/veui/src/components/Form/Fieldset.vue | 10 +++++++++-
 2 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/packages/veui/src/components/Form/Field.vue b/packages/veui/src/components/Form/Field.vue
index 4ed086a3a..78ab5e856 100644
--- a/packages/veui/src/components/Form/Field.vue
+++ b/packages/veui/src/components/Form/Field.vue
@@ -9,16 +9,16 @@
     'veui-field-required': isRequired
   }"
 >
-  <span v-if="label || $slots.label" class="veui-form-label">
+  <div v-if="label || $slots.label" class="veui-form-label">
     <slot name="label"><veui-label>{{ label }}</veui-label></slot>
-  </span>
+  </div>
   <slot/>
-  <span v-if="tip || $slots.tip" class="veui-form-tip"><slot name="tip">{{ tip }}</slot></span>
-  <p
+  <div v-if="tip || $slots.tip" class="veui-form-tip"><slot name="tip">{{ tip }}</slot></div>
+  <div
     v-if="!validity.valid && !!validity.message"
     class="veui-field-error"
     :title="validity.message"
-  ><veui-icon :name="icons.alert"/>{{ validity.message }}</p>
+  ><veui-icon :name="icons.alert"/>{{ validity.message }}</div>
 </div>
 </template>
 
diff --git a/packages/veui/src/components/Form/Fieldset.vue b/packages/veui/src/components/Form/Fieldset.vue
index bd861b99f..6f0b5403f 100644
--- a/packages/veui/src/components/Form/Fieldset.vue
+++ b/packages/veui/src/components/Form/Fieldset.vue
@@ -1,5 +1,13 @@
 <template>
-<veui-field :ui="ui" ref="field" class="veui-fieldset" :class="{'veui-fieldset-required': required}" v-bind="attrs">
+<veui-field
+  :ui="ui"
+  ref="field"
+  class="veui-fieldset"
+  :class="{
+    'veui-fieldset-required': required
+  }"
+  role="group"
+  v-bind="attrs">
   <template v-if="$slots.label" slot="label"><slot name="label"/></template>
   <slot/>
   <template v-if="$slots.tip" slot="tip"><slot name="tip"/></template>