Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

lazyload_images should be enabled for IE6/IE7 #581

Closed
GoogleCodeExporter opened this issue Apr 6, 2015 · 4 comments
Closed

lazyload_images should be enabled for IE6/IE7 #581

GoogleCodeExporter opened this issue Apr 6, 2015 · 4 comments

Comments

@GoogleCodeExporter
Copy link

We were disabling it for those browsers in sympathy with the limited 
image-inlining support.  However we don't need to inline large images for 
lazyload; just a small blank one, so it should work fine.

Original issue reported on code.google.com by jmara...@google.com on 30 Nov 2012 at 10:20

@GoogleCodeExporter
Copy link
Author

Actually IE7 does handle the base64 image currently needed for lazyload images. 
 However, we should serve that blank image instead from 
/mod_pagespeed_static/1.gif .  That would allow us to turn this filter on for 
IE6 & IE7.

To accomplish this, we can factor out and generalize StaticJavascriptManager.

Original comment by jmara...@google.com on 4 Dec 2012 at 5:56

@GoogleCodeExporter
Copy link
Author

To clarify, IE6 and IE7 do not handle the inlined base64 image. There is a 
really terrible looking hack using MHTML to get this to work, see 
http://www.phpied.com/mhtml-when-you-need-data-uris-in-ie7-and-under/, but I 
agree a better solution is to use /mod_pagespeed_static/1.gif.

Original comment by j...@google.com on 4 Dec 2012 at 5:59

@GoogleCodeExporter
Copy link
Author

Original comment by jmara...@google.com on 28 Jan 2013 at 7:20

@GoogleCodeExporter
Copy link
Author

Resolved by r2486. The blank image is no longer inlined but instead served from 
/mod_pagespeed_static. The lazyload JS has been tested with IE6 and IE7 as well.

Original comment by j...@google.com on 12 Feb 2013 at 7:11

  • Changed state: Fixed
  • Added labels: Milestone-v25, release-note

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant