-
Notifications
You must be signed in to change notification settings - Fork 6.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
text color md-fab and md-mini-fab is not the default-contrast color #2539
Comments
Even so, when I add a class in my custom-style.scss, the md-contast color is overwritten. This is my custom-style.scss: @include md-core(); $primary: md-palette($md-grey); $theme: md-light-theme($primary, $accent); .accent-text-color{ md-list-item.zoekresultaat:hover{ @include angular-material-theme($theme);` I can use all this classes in my whole application, but the contast color always is overwritten. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Bug, feature request, or proposal:
Bug
What is the expected behavior?
The text color of a md-fab or md-mini-fab should be the default-contrast of the chosen theme accent color. For example when using the prebuilt the indigo-pink the default $accent is set to A200, so according to the palette the contrast A200 is white, so the text-color of the md-fab or md-mini-fab should be white against a pink background-color.
What is the current behavior?
The text-color is set to the current color (in my case grey).
What are the steps to reproduce?
Providing a Plunker (or similar) is the best way to get the team to see your issue.
Plunker template: http://plnkr.co/edit/LbeegL?p=preview
http://stackoverflow.com/questions/40478419/angular-material2-button-text-color
What is the use-case or motivation for changing an existing behavior?
readability
Which versions of Angular, Material, OS, browsers are affected?
I'm using angular-material 2.0.0-beta.1
Is there anything else we should know?
No
The text was updated successfully, but these errors were encountered: