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

IE 7+8 only renders small image? #33

Closed
floq-design opened this Issue Jul 19, 2012 · 5 comments

Comments

Projects
None yet
6 participants
@floq-design

Hi
I'm testing the demo files that come with the project and in IE 9 (in IE 7 + 8 modes) it's only showing the small image.. is this correct behaviour?

EDIT:

Just realised the demo file an HTML5 doctype.. but - just added the head from HTML5 boilerplate and still no go..

EDIT v2:

I think I'm being really stupid and just not amending the src for the default image..

@nring

This comment has been minimized.

Show comment
Hide comment
@nring

nring Aug 14, 2012

Requesting this to be reopened. I'm unable to get anything but the small default image on IE7 and IE8. I'm seeing the same bug on the official demo page (http://scottjehl.github.com/picturefill/) with different machines running IE8 as well.

Edit:
After some more digging, it seems like this is a known MatchMedia issue. I resolved it by updating my local version of MatchMedia to the one forked here: https://github.com/benschwarz/matchMedia.js/blob/IE7-8/matchMedia.js

nring commented Aug 14, 2012

Requesting this to be reopened. I'm unable to get anything but the small default image on IE7 and IE8. I'm seeing the same bug on the official demo page (http://scottjehl.github.com/picturefill/) with different machines running IE8 as well.

Edit:
After some more digging, it seems like this is a known MatchMedia issue. I resolved it by updating my local version of MatchMedia to the one forked here: https://github.com/benschwarz/matchMedia.js/blob/IE7-8/matchMedia.js

@baamenabar

This comment has been minimized.

Show comment
Hide comment
@baamenabar

baamenabar Oct 23, 2012

This workaround could be annotated in the documentation, or even made into a branch.

This workaround could be annotated in the documentation, or even made into a branch.

@scottjehl

This comment has been minimized.

Show comment
Hide comment
@scottjehl

scottjehl Oct 23, 2012

Owner

There's a workaround documented in the readme which works well. https://github.com/scottjehl/picturefill#supporting-ie-desktop

Aside from that, you can try the MatchMedia polyfill fork that includes IE support.

Fixes #33

On Oct 23, 2012, at 5:22 PM, Agustín Amenabar wrote:

This workaround could be annotated in the documentation, or even made into a branch.


Reply to this email directly or view it on GitHub.

Owner

scottjehl commented Oct 23, 2012

There's a workaround documented in the readme which works well. https://github.com/scottjehl/picturefill#supporting-ie-desktop

Aside from that, you can try the MatchMedia polyfill fork that includes IE support.

Fixes #33

On Oct 23, 2012, at 5:22 PM, Agustín Amenabar wrote:

This workaround could be annotated in the documentation, or even made into a branch.


Reply to this email directly or view it on GitHub.

@adick

This comment has been minimized.

Show comment
Hide comment
@adick

adick Jul 29, 2014

Yep, unfortunately in IE8 always the small image is displayed. (picture tag version)

@scottjehl which workaround do you mean? The link does not show a workaround anymore.

Also, when I use the other matchMedia polyfill @nring poiinted out, respond.js does not work anymore.

picturefill-ie8

adick commented Jul 29, 2014

Yep, unfortunately in IE8 always the small image is displayed. (picture tag version)

@scottjehl which workaround do you mean? The link does not show a workaround anymore.

Also, when I use the other matchMedia polyfill @nring poiinted out, respond.js does not work anymore.

picturefill-ie8

@benplum

This comment has been minimized.

Show comment
Hide comment
@benplum

benplum Sep 10, 2014

This is very much still an issue. Any word on a fix (at least ie8)?

benplum commented Sep 10, 2014

This is very much still an issue. Any word on a fix (at least ie8)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment