From 2b7007f38e8734307b22d639911296f6f2e16d4b Mon Sep 17 00:00:00 2001
From: "Manu Mtz.-Almeida" <manu.mtza@gmail.com>
Date: Tue, 7 Aug 2018 20:27:57 +0200
Subject: [PATCH] fix(all): buttons are type="button"

---
 core/src/components/action-sheet/action-sheet.tsx     | 3 ++-
 core/src/components/alert/alert.tsx                   | 4 +++-
 core/src/components/back-button/back-button.tsx       | 1 +
 core/src/components/menu-button/menu-button.tsx       | 2 +-
 core/src/components/picker-column/picker-column.tsx   | 1 +
 core/src/components/picker/picker.tsx                 | 1 +
 core/src/components/segment-button/segment-button.tsx | 1 +
 7 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/core/src/components/action-sheet/action-sheet.tsx b/core/src/components/action-sheet/action-sheet.tsx
index deb9e3de56d..bbb823b1e1d 100644
--- a/core/src/components/action-sheet/action-sheet.tsx
+++ b/core/src/components/action-sheet/action-sheet.tsx
@@ -237,7 +237,7 @@ export class ActionSheet implements OverlayInterface {
               </div>
             }
             {buttons.map(b =>
-              <button class={buttonClass(b)} onClick={() => this.buttonClick(b)}>
+              <button type="button" class={buttonClass(b)} onClick={() => this.buttonClick(b)}>
                 <span class="action-sheet-button-inner">
                   {b.icon && <ion-icon icon={b.icon} lazy={false} class="action-sheet-icon" />}
                   {b.text}
@@ -249,6 +249,7 @@ export class ActionSheet implements OverlayInterface {
           {cancelButton &&
             <div class="action-sheet-group action-sheet-group-cancel">
               <button
+                type="button"
                 class={buttonClass(cancelButton)}
                 onClick={() => this.buttonClick(cancelButton)}
               >
diff --git a/core/src/components/alert/alert.tsx b/core/src/components/alert/alert.tsx
index 08bc99dc7b6..106b14f606a 100644
--- a/core/src/components/alert/alert.tsx
+++ b/core/src/components/alert/alert.tsx
@@ -315,6 +315,7 @@ export class Alert implements OverlayInterface {
       <div class="alert-checkbox-group" aria-labelledby={labelledby}>
         { inputs.map(i => (
           <button
+            type="button"
             onClick={() => this.cbClick(i)}
             aria-checked={i.checked ? 'true' : null}
             id={i.id}
@@ -345,6 +346,7 @@ export class Alert implements OverlayInterface {
       <div class="alert-radio-group" role="radiogroup" aria-labelledby={labelledby} aria-activedescendant={this.activeId}>
         { inputs.map(i => (
           <button
+            type="button"
             onClick={() => this.rbClick(i)}
             aria-checked={i.checked ? 'true' : null}
             disabled={i.disabled}
@@ -413,7 +415,7 @@ export class Alert implements OverlayInterface {
     return (
       <div class={alertButtonGroupClass}>
         {buttons.map(button =>
-          <button class={buttonClass(button)} tabIndex={0} onClick={() => this.buttonClick(button)}>
+          <button type="button" class={buttonClass(button)} tabIndex={0} onClick={() => this.buttonClick(button)}>
             <span class="alert-button-inner">
               {button.text}
             </span>
diff --git a/core/src/components/back-button/back-button.tsx b/core/src/components/back-button/back-button.tsx
index f5c57d89163..4a4636fbe9d 100644
--- a/core/src/components/back-button/back-button.tsx
+++ b/core/src/components/back-button/back-button.tsx
@@ -77,6 +77,7 @@ export class BackButton {
 
     return (
       <button
+        type="button"
         class="back-button-native"
         onClick={ev => this.onClick(ev)}>
         <span class="back-button-inner">
diff --git a/core/src/components/menu-button/menu-button.tsx b/core/src/components/menu-button/menu-button.tsx
index 9a8ef9bc3ac..df1a2b4782e 100644
--- a/core/src/components/menu-button/menu-button.tsx
+++ b/core/src/components/menu-button/menu-button.tsx
@@ -49,7 +49,7 @@ export class MenuButton {
     const menuIcon = this.config.get('menuIcon', 'menu');
     return (
       <ion-menu-toggle menu={this.menu} autoHide={this.autoHide}>
-        <button>
+        <button type="button">
           <slot>
             <ion-icon icon={menuIcon} mode={this.mode} color={this.color} lazy={false} />
           </slot>
diff --git a/core/src/components/picker-column/picker-column.tsx b/core/src/components/picker-column/picker-column.tsx
index a82eaf586a1..20314afa0fb 100644
--- a/core/src/components/picker-column/picker-column.tsx
+++ b/core/src/components/picker-column/picker-column.tsx
@@ -330,6 +330,7 @@ export class PickerColumnCmp {
         ref={ el => this.optsEl = el }>
         { col.options.map((o, index) =>
           <Button
+            type="button"
             class={{ 'picker-opt': true, 'picker-opt-disabled': !!o.disabled }}
             disable-activated
             opt-index={index}>
diff --git a/core/src/components/picker/picker.tsx b/core/src/components/picker/picker.tsx
index cac7959e1a0..a82e4ef46ab 100644
--- a/core/src/components/picker/picker.tsx
+++ b/core/src/components/picker/picker.tsx
@@ -283,6 +283,7 @@ export class Picker implements OverlayInterface {
           {buttons.map(b => (
             <div class={buttonWrapperClass(b)}>
               <button
+                type="button"
                 onClick={() => this.buttonClick(b)}
                 class={buttonClass(b)}
               >
diff --git a/core/src/components/segment-button/segment-button.tsx b/core/src/components/segment-button/segment-button.tsx
index cdf8bf8425a..88b04899620 100644
--- a/core/src/components/segment-button/segment-button.tsx
+++ b/core/src/components/segment-button/segment-button.tsx
@@ -69,6 +69,7 @@ export class SegmentButton {
   render() {
     return [
       <button
+        type="button"
         aria-pressed={this.checked ? 'true' : null}
         class="segment-button-native"
         disabled={this.disabled}