fix(VExpansionPanel): set proper button type in header

fixes #7471
vuetifyjs · Jun 12, 2019 · cf0da4b · cf0da4b
1 parent eb7d5b3
commit cf0da4b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/packages/vuetify/src/components/VExpansionPanel/VExpansionPanelHeader.ts b/packages/vuetify/src/components/VExpansionPanel/VExpansionPanelHeader.ts
@@ -101,6 +101,7 @@ export default baseMixins.extend<options>().extend({
       class: this.classes,
       attrs: {
         tabindex: this.isDisabled ? -1 : null,
+        type: 'button',
       },
       directives: [{
         name: 'ripple',

diff --git a/...etify/src/components/VExpansionPanel/__tests__/__snapshots__/VExpansionPanel.spec.ts.snap b/...etify/src/components/VExpansionPanel/__tests__/__snapshots__/VExpansionPanel.spec.ts.snap
@@ -5,7 +5,9 @@ exports[`VExpansionPanel should hide actions and match snapshot 1`] = `
 <div aria-expanded="false"
      class="v-expansion-panel"
 >
-  <button class="v-expansion-panel-header">
+  <button type="button"
+          class="v-expansion-panel-header"
+  >
   </button>
   <div class="v-expansion-panel-content"
        style="display: none;"
@@ -20,7 +22,9 @@ exports[`VExpansionPanel should hide actions and match snapshot 2`] = `
 <div aria-expanded="false"
      class="v-expansion-panel"
 >
-  <button class="v-expansion-panel-header">
+  <button type="button"
+          class="v-expansion-panel-header"
+  >
     <div class="v-expansion-panel-header__icon">
       <i aria-hidden="true"
          class="v-icon material-icons theme--light"