-
Notifications
You must be signed in to change notification settings - Fork 400
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.
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
Sidebar issue on IE11 #18
Comments
@Bodom78 Let me know if this works for you on IE11 |
Links work fine in IE, the styling issue remains. Looks like the vertical text alignment on a list item in IE can be fixed by adding a height to md-list-item as discussed at angular/material#3764 Testing in the DOM Explorer confirms this works but it is really a hack just for IE to work. Only problem would be if you had 3 lines of text on a menu item which is unlikely. |
Oh okay.. that's great Thanks a lot for your help :D |
@Bodom78 let me know if it fixes it, so I can close the issue |
Looks good and working fine in IE11 now. |
Great, thanks! |
Originally posted by @Bodom78 - issue #16
I have some workarounds in the
sidebar.html
code for theui-sref
to work.. So this might be causing an issue on IE11..But then I noticed that this is an upstream issue on angular/material repo:
angular/material#2131
I'm pushing a temporary fix as mentioned in this comment until the upstream issue is resolved.
The text was updated successfully, but these errors were encountered: