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

Padding on item-list with ng-click wrong on ie 11 #7557

Closed
gpopovic opened this issue Mar 14, 2016 · 6 comments
Closed

Padding on item-list with ng-click wrong on ie 11 #7557

gpopovic opened this issue Mar 14, 2016 · 6 comments
Assignees
@devversion
Labels
pr: merge ready This PR is ready for a caretaker to review ui: flexbox ui: theme
Milestone
1.1.0

Comments

@gpopovic
Copy link

On ie11, padding is wrong ...
ebb2b408-af11-4f9f-a786-99b293c3e12f
@devversion
Copy link
Member

Provide a demo, and check within master. In addition to that try confirming with #7531

@devversion devversion added the needs: demo A CodePen demo or GitHub repository is needed to demonstrate the reproduction of the issue label Mar 14, 2016
@devversion devversion self-assigned this Mar 14, 2016
@gpopovic
Copy link
Author

@devversion check in IE11

http://codepen.io/anon/pen/ZWpwRa

@ThomasBurleson ThomasBurleson modified the milestone: 1.1.0 Mar 14, 2016
@devversion devversion added needs: investigation The cause of this issue is not well understood and needs to be investigated by the team or community and removed needs: demo A CodePen demo or GitHub repository is needed to demonstrate the reproduction of the issue labels Mar 14, 2016
@devversion
Copy link
Member

Seems like the vertical alignment for the md-list-item-inner div isn't working probably on IE11

IE11:
image

Chrome:
image

@ThomasBurleson ThomasBurleson added ui: flexbox and removed needs: investigation The cause of this issue is not well understood and needs to be investigated by the team or community labels Mar 14, 2016
devversion added a commit to devversion/material that referenced this issue Mar 15, 2016
@devversion
fix(list): fix compatibility with IE11
b2c5e90 
* Added a mixin / workaround for the flexbox bug for Internet Explorer 11.

Fixes angular#7557.
@devversion devversion mentioned this issue Mar 15, 2016
Closed
@devversion devversion added the pr: merge ready This PR is ready for a caretaker to review label Mar 16, 2016
@gpopovic
Copy link
Author

Ok, so this actually isn't fixed.

It works well on IE 11 Version 11.162.10586.0 (on Windows 10)
but it doesn't work on IE 11.0.9600 Version (on Windows 7)...as fix never exited

@devversion sorry man :D I know this must be frustrating

@devversion
Copy link
Member

Damn... 😄 I thought now everything is great with the list component. Can you post a screenshot of the Dom Inspection / Preview Output etc. So I can confirm, that the bug is still present?

@gpopovic
Copy link
Author

work

@devversion Please note, it works on IE11 on Win 10.

This is the picture on IE11 Windows 7.
I'm having some problems with IE on home computer, so i can't check dom..i'll send you one in few days(I checked on multiple win7 computes on IE 11, always the same error)

@gpopovic gpopovic mentioned this issue Mar 18, 2016
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@devversion devversion

Labels
pr: merge ready This PR is ready for a caretaker to review ui: flexbox ui: theme
Projects
None yet
Milestone
1.1.0
Development

Successfully merging a pull request may close this issue.

fix(list): fix compatibility with IE11 devversion/material
3 participants
@ThomasBurleson @gpopovic @devversion