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

mdListItem: Secondary clickable md-icon needs focus style #2340

Closed
marcysutton opened this issue Apr 15, 2015 · 0 comments
Closed

mdListItem: Secondary clickable md-icon needs focus style #2340

marcysutton opened this issue Apr 15, 2015 · 0 comments
Assignees
Milestone

Comments

@marcysutton
Copy link
Contributor

The new focus styles in #2273 and #1594 require JS to set focus only on true keyboard interactions (hacking around the browser default of persisting focus on click). This behavior is not implemented on md-icon, but on md-button. To minimize duplication of code, mdList's secondary md-icon directives with ng-click on them should really be md-button.

To fix this, we can either transform md-icon into md-button behind the scenes or change the HTML API to use <md-button ng-click""><md-icon /></md-button>.

@marcysutton marcysutton changed the title mdList: Secondary clickable md-icon needs focus style mdListItem: Secondary clickable md-icon needs focus style Apr 15, 2015
@marcysutton marcysutton added this to the 0.9.0 milestone Apr 16, 2015
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