Skip to content
This repository has been archived by the owner on Aug 29, 2023. It is now read-only.

mdListItem: Layer buttons so highlight covers whole row #2222

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

mdListItem: Layer buttons so highlight covers whole row #2222

marcysutton opened this issue Apr 9, 2015 · 0 comments
Assignees
@marcysutton
Milestone
0.9.0

Comments

@marcysutton
Copy link
Contributor

The new list items have secondary controls sitting next to the primary ones, instead of on top, so the highlight for the primary action doesn't cover the whole row. An easy fix would be to position the secondary control on top of the primary.

Message A checkbox with comment icon button

There is still work to be done for the correct focus styles (no focus on click, focus switches/checkboxes on keyboard tab), but those are tracked in separate issues/PRs. #1594 #1336
@marcysutton marcysutton self-assigned this Apr 9, 2015
@marcysutton marcysutton added this to the 0.9.0 milestone Apr 9, 2015
marcysutton pushed a commit that referenced this issue Apr 10, 2015
fix(list): focus covers whole row, IE11 layout fix
8dd26b8 
Closes #2222
@ajoslin ajoslin added the in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs label Apr 10, 2015
@marcysutton marcysutton mentioned this issue Apr 10, 2015
Closed
@marcysutton marcysutton added resolution: fixed and removed in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs labels Apr 10, 2015
marcysutton pushed a commit that referenced this issue Apr 10, 2015
fix(list): focus covers whole row, IE11 layout fix
4a57265 
Closes #2222
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@marcysutton marcysutton

Labels
None yet
Projects
None yet
Milestone
0.9.0
Development

Successfully merging a pull request may close this issue.

fix(list): focus covers whole row, IE11 layout fix angular/material
2 participants
@marcysutton @ajoslin