Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

md-button(not raised): No background color on hover? #2382

Closed
epelc opened this issue Apr 17, 2015 · 6 comments
Closed

md-button(not raised): No background color on hover? #2382

epelc opened this issue Apr 17, 2015 · 6 comments
Assignees
Milestone

Comments

@epelc
Copy link
Contributor

epelc commented Apr 17, 2015

It seems that the slight grey background for non raised buttons has been removed. However this doesn't conform with the spec.

http://www.google.com/design/spec/components/buttons.html#buttons-flat-raised-buttons

In 0.9.0-rc1 it currently looks like normal when it should look like hover with a slight grey background.
image

@epelc
Copy link
Contributor Author

epelc commented Apr 17, 2015

It also isn't showing when using md-noink as you can see in the buttons demo. I would think it's the same cause though.

https://material.angularjs.org/#/demo/material.components.button

@marcysutton
Copy link
Contributor

We were actually going off of internal feedback that said "Buttons: we don't have hover states." I have followed up to clarify.

@epelc
Copy link
Contributor Author

epelc commented Apr 17, 2015

@marcysutton As a user it seems confusing when there isn't a hover state. The only visual cue I get that something is a button then is that the text is all caps. Also not having a hover state seems to go against having a focused state.

@marcysutton
Copy link
Contributor

After looking closer at it, I totally agree. It's less clear with flat buttons that they are indeed buttons without some sort of visual feedback.

@epelc
Copy link
Contributor Author

epelc commented Apr 17, 2015

@marcysutton Thanks for the fix!

@ThomasBurleson
Copy link
Contributor

@marcysutton - 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants