fix(menu-button): Not visible if toolbar has primary color (#15847)

* fix(menu-button): Correct color if toolbar has custom color * fix(menu-button): Fix variable name
ionic-team · Oct 8, 2018 · e2ea08b · e2ea08b
1 parent 4d750ed
commit e2ea08b
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
diff --git a/core/src/components/menu-button/menu-button.ios.scss b/core/src/components/menu-button/menu-button.ios.scss
@@ -5,7 +5,7 @@
 // --------------------------------------------------
 
 :host {
-  color: #{ion-color(primary, base)};
+  --color: #{$menu-button-ios-color};
 }
 
 :host(.activated) {

diff --git a/core/src/components/menu-button/menu-button.md.scss b/core/src/components/menu-button/menu-button.md.scss
@@ -3,6 +3,9 @@
 
 // MD Menu Button
 // --------------------------------------------------
+:host {
+  --color: #{$menu-button-md-color};
+}
 
 button {
   @include padding(0, 8px);

diff --git a/core/src/components/menu-button/menu-button.scss b/core/src/components/menu-button/menu-button.scss
@@ -4,6 +4,8 @@
 // --------------------------------------------------
 
 :host {
+  color: var(--color);
+
   pointer-events: all;
 
   text-align: center;
@@ -52,3 +54,10 @@ ion-icon {
 
   pointer-events: none;
 }
+
+// Menu Button with Color
+// --------------------------------------------------
+
+:host(.ion-color) .button-native {
+  color: current-color(base);
+}