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

Focus style doesn't match material spec #111

Closed
cleercode opened this issue Apr 13, 2016 · 2 comments
Closed

Focus style doesn't match material spec #111

cleercode opened this issue Apr 13, 2016 · 2 comments

Comments

@cleercode
Copy link

Expected outcome

Focusing the button using the tab key should give the button a background color of either the fill color or black (unclear from the wording of the spec) at 12% opacity: https://www.google.com/design/spec/components/buttons.html

Actual outcome

Focusing the button makes the font bold.

Note: The bold font weight is especially difficult to distinguish from the normal state if the normal state is made to use medium instead of regular font weight, as the material spec also specifies.

Live Demo

https://elements.polymer-project.org/elements/paper-button?view=demo:demo/index.html

@cleercode
Copy link
Author

Should have searched first: #94 #95

That said, it'd be good to see some consistency on this between Polymer and MD. Apps that are trying to properly implement the spec will override the normal font weight to medium, which will make using bold for focus state insufficient. See for example Chrome's settings UI (Canary, chrome://md-settings).

@keanulee
Copy link
Contributor

#95 pretty much sums up why we haven't done this. For now we suggest you override it if that's what you want (knowing why we did this in the first place).

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

2 participants