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

md-list-item with href or ng-click have different margin #6317

Closed
XGHeaven opened this issue Dec 15, 2015 · 1 comment
Closed

md-list-item with href or ng-click have different margin #6317

XGHeaven opened this issue Dec 15, 2015 · 1 comment
Labels
needs: investigation The cause of this issue is not well understood and needs to be investigated by the team or community P1: urgent Urgent issues that should be addressed in the next minor or patch release. ui: CSS

Comments

@XGHeaven
Copy link

md-list-item with href have external margin value.
image
md-list-item with ng-click will create a button element.
That element with md-button.md-no-style to remove margin value.
image
But it's with href doesn't remove it.

I suppose add a css called a.md-no-style to remove margin value inherit from md-button
@ripper2hl
Copy link

Same problem with ng-href on md-list. 👍 check this :

md-list-item

devversion added a commit to devversion/material that referenced this issue Dec 29, 2015
@devversion
fix(list): fix list margin if anchor is present.
995bb53 
If there is currently a `md-button` anchor, we should remove the margin to order the items as the other items.

Fixes angular#6317
This was referenced Dec 29, 2015
Closed
Closed
devversion added a commit to devversion/material that referenced this issue Jan 5, 2016
@devversion
fix(list): fix list margin if anchor is present.
d9d44fc 
If there is currently a `md-button` anchor, we should remove the margin to order the items as the other items.

Fixes angular#6317
devversion added a commit to devversion/material that referenced this issue Jan 5, 2016
@devversion
fix(list): fix list margin if anchor is present.
e704eed 
If there is currently a `md-button` anchor, we should remove the margin to order the items as the other items.

Fixes angular#6317
@topherfangio topherfangio added ui: CSS needs: investigation The cause of this issue is not well understood and needs to be investigated by the team or community priority: medium labels Jan 6, 2016
@topherfangio topherfangio added this to the 1.1.0 milestone Jan 6, 2016
@ThomasBurleson ThomasBurleson added P1: urgent Urgent issues that should be addressed in the next minor or patch release. and removed P1: urgent Urgent issues that should be addressed in the next minor or patch release. labels Jan 15, 2016
@ThomasBurleson ThomasBurleson modified the milestones: 1.1.0, Backlog Jan 15, 2016
devversion added a commit to devversion/material that referenced this issue Feb 22, 2016
@devversion
fix(list): fix list margin if anchor is present.
bfa702e 
If there is currently a `md-button` anchor, we should remove the margin to order the items as the other items.

Fixes angular#5608 Fixes angular#6317
@JasonTheProgrammer JasonTheProgrammer mentioned this issue May 29, 2016
Closed
@Splaktar Splaktar removed this from the - Backlog milestone Feb 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
needs: investigation The cause of this issue is not well understood and needs to be investigated by the team or community P1: urgent Urgent issues that should be addressed in the next minor or patch release. ui: CSS
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@topherfangio @ThomasBurleson @ripper2hl @Splaktar @XGHeaven