Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add Hover, Focus, and Pressed states to MDC Card #1709

Closed
lynnmercier opened this issue Dec 7, 2017 · 2 comments
Closed

Add Hover, Focus, and Pressed states to MDC Card #1709

lynnmercier opened this issue Dec 7, 2017 · 2 comments
Labels
backlog

Comments

@lynnmercier
Copy link
Contributor

MDC Cards should be clickable, and should use our states subsystem to show hover, focus, and pressed states
@kfranqueiro
Copy link
Contributor

Is this something that should happen for all cards, or should it be either opt-in or opt-out? (e.g. I'm planning to add an opt-out class for mdc-list-item when I cover lists.)

@lynnmercier lynnmercier changed the title Add Hover, Focus, and Pressed states of MDC Card Add Hover, Focus, and Pressed states to MDC Card Jan 19, 2018
@lynnmercier
Copy link
Contributor Author

As discussed in sprint planning, it will be optional. An opt-in.

@acdvorak acdvorak mentioned this issue Jan 23, 2018
Merged
acdvorak added a commit that referenced this issue Feb 6, 2018
@acdvorak
feat(card): Add primary action element with hover, focused, and press…
3949dbe 
…ed states (#2039)

Fixes #1709
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
backlog
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kfranqueiro @lynnmercier