From d22f04bd900a1825b9d81d97021efcbf96280a81 Mon Sep 17 00:00:00 2001
From: "Manu Mtz.-Almeida" <manu.mtza@gmail.com>
Date: Tue, 24 Jul 2018 02:38:55 +0200
Subject: [PATCH] fix(menu-button): fix sass linting

---
 core/src/components/alert/alert-interface.ts         | 2 +-
 core/src/components/item-option/item-option.tsx      | 1 +
 core/src/components/item-options/item-options.tsx    | 3 +--
 core/src/components/menu-button/menu-button.ios.scss | 2 +-
 core/src/components/menu-button/menu-button.md.scss  | 2 +-
 core/src/components/menu-button/menu-button.scss     | 9 ++++++---
 6 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/core/src/components/alert/alert-interface.ts b/core/src/components/alert/alert-interface.ts
index b27a7d630a..3af7eaa03e 100644
--- a/core/src/components/alert/alert-interface.ts
+++ b/core/src/components/alert/alert-interface.ts
@@ -13,7 +13,7 @@ export interface AlertOptions {
 
 export interface AlertInput {
   type: string;
-  name: string | number;
+  name: string;
   placeholder?: string;
   value?: string;
   label?: string;
diff --git a/core/src/components/item-option/item-option.tsx b/core/src/components/item-option/item-option.tsx
index 1b30797485..a10c650ffc 100644
--- a/core/src/components/item-option/item-option.tsx
+++ b/core/src/components/item-option/item-option.tsx
@@ -60,6 +60,7 @@ export class ItemOption {
 
     return (
       <TagType
+        type="button"
         class="item-option-native"
         disabled={this.disabled}
         href={this.href}
diff --git a/core/src/components/item-options/item-options.tsx b/core/src/components/item-options/item-options.tsx
index 4520d4e014..00aed1b3c5 100644
--- a/core/src/components/item-options/item-options.tsx
+++ b/core/src/components/item-options/item-options.tsx
@@ -12,8 +12,7 @@ import { isEndSide } from '../../utils/helpers';
 export class ItemOptions {
   @Element() el!: HTMLElement;
 
-  @Prop({ context: 'window' })
-  win!: Window;
+  @Prop({ context: 'window' }) win!: Window;
 
   /**
    * The side the option button should be on. Possible values: `"start"` and `"end"`. Defaults to `"end"`. If you have multiple `ion-item-options`, a side must be provided for each.
diff --git a/core/src/components/menu-button/menu-button.ios.scss b/core/src/components/menu-button/menu-button.ios.scss
index fcf4a93cd4..b47bb58647 100644
--- a/core/src/components/menu-button/menu-button.ios.scss
+++ b/core/src/components/menu-button/menu-button.ios.scss
@@ -9,7 +9,7 @@
 }
 
 button {
-  padding: 0 5px;
+  @include padding(0, 5px);
 }
 
 ion-icon {
diff --git a/core/src/components/menu-button/menu-button.md.scss b/core/src/components/menu-button/menu-button.md.scss
index e0fb8a1c25..610a6c9513 100644
--- a/core/src/components/menu-button/menu-button.md.scss
+++ b/core/src/components/menu-button/menu-button.md.scss
@@ -9,7 +9,7 @@
 }
 
 button {
-  padding: 0 8px;
+  @include padding(0, 8px);
 }
 
 ion-icon {
diff --git a/core/src/components/menu-button/menu-button.scss b/core/src/components/menu-button/menu-button.scss
index 17378347a9..4ce6b215f0 100644
--- a/core/src/components/menu-button/menu-button.scss
+++ b/core/src/components/menu-button/menu-button.scss
@@ -15,7 +15,7 @@ button {
   @include font-smoothing();
 
   display: flex;
-  height: 32px;
+
   position: relative;
 
   flex-flow: row nowrap;
@@ -23,6 +23,8 @@ button {
   align-items: center;
   justify-content: center;
 
+  height: 32px;
+
   border: 0;
 
   outline: none;
@@ -47,7 +49,8 @@ button {
 }
 
 ion-icon {
-  padding: 0;
-  margin: 0;
+  @include margin(0);
+  @include padding(0);
+
   pointer-events: none;
 }