Skip to content
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

img-responsive images do not render when placed inside a list item of an inline list in IE8 #14214

Closed
tommedema opened this issue Jul 23, 2014 · 4 comments

Comments

@tommedema
Copy link

Steps to reproduce:

  1. create an inline list: <ul class="list-inline"></ul>
  2. add a (PNG) image as a list item with img-responsive class: <ul class="list-inline"><li><img class="img-responsive" height="52" width="130" src="https://www.whataddress.com/img/company_logo_bol.png" alt="bol.com" /></li></ul>

Expected behavior: the image should be displayed

Actual behavior: the image will not render in IE8

Note that removing list-inline or img-responsive will cause the image to render again, but when keeping img-responsive the image will cover the entire screen, which is undesired.

You can reproduce it at (http://jsfiddle.net/tommedema/PNqzH/7/) (editor in your modern browser) and (http://jsfiddle.net/tommedema/PNqzH/8/embedded/result/) (use this in IE8 to display it)

@carasmo
Copy link

carasmo commented Jul 23, 2014

http://stackoverflow.com/a/17489727/1004312
apparently... IE8 does support the max-width attribute but only when given a defined width first.

.selector {
    width: 100%;
    max-width: 250px;
}

@cvrebert cvrebert added this to the v3.2.1 milestone Jul 23, 2014
@cvrebert
Copy link
Collaborator

What's the point of setting height="52" width="130" if you're making the image responsively scale anyway?

@cvrebert cvrebert removed this from the v3.2.1 milestone Jul 23, 2014
@mdo
Copy link
Member

mdo commented Jul 24, 2014

Almost inclined to punt on this, it's an odd one. Sounds like h5bp/html5-boilerplate#984 (comment) maybe?

@mdo
Copy link
Member

mdo commented Aug 2, 2014

Punting.

@mdo mdo closed this as completed Aug 2, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants