search

louisremi / background-size-polyfill

Adds support for background-size "cover" and "contain" to IE8
http://louisremi.github.com/background-size-polyfill/
MIT License
1.26k stars 359 forks source link

polyfill only works if content doesn't overflow screen #43

Open davidrhoden opened 10 years ago

davidrhoden commented 10 years ago

My problem is related to "Doesn't work" but is a little different. The polyfill works fine if the content on the page is short. if the content overflows the screen, however, the polyfill either doesn't work, or is mispositioned too far to the left. Occasionally a brief flash of the correctly positioned image can be seen before the image adopts its incorrect position. The pages involved mostly have static contents, but one page does have content that is added as an iframe.

The site is live: http://hurrayfortheriffraff.com if you would like to see these issues in IE8. Everything works fine in Chrome/afari/Firefox, on my Mac anyway.

jefferyto commented 10 years ago

I don't see any background images in IE8... According to the developer tools, the background declarations are being "included" into the filter declarations (in the Style tab, you'll see that the background declarations don't appear as toggle-able styles, but instead at the right end of the filter declarations). I suggest removing all of the filter declarations before we try to find out what's happening with the polyfill.

(I'm guessing you added the filter declarations when the polyfill didn't appear to work; note that with the AlphaImageLoader filter, the image paths are relative to the HTML page, not the CSS, so those declarations aren't working either.)

davidrhoden commented 10 years ago

Thanks for answering! The site is live, so I ended up having to remove the images so it's at least presentable. I tried the filters first, and wasn't sure if I still needed them. I'll try removing them and see what that does. Is the polyfill supposed to support IE7?

David Rhoden 2755 Orchid Street, Apt. B New Orleans, Louisiana 70119 504-812-8475 david@davidrhoden.com http://davidrhoden.com http://thestackswebsite.com http://thisismyhappening.com

On Feb 4, 2014, at 11:58 AM, Jeffery To notifications@github.com wrote:

I don't see any background images in IE8... According to the developer tools, the background declarations are being "included" into the filter declarations (in the Style tab, you'll see that the background declarations don't appear as toggle-able styles, but instead at the right end of the filter declarations). I suggest removing all of the filter declarations before we try to find out what's happening with the polyfill.

(I'm guessing you added the filter declarations when the polyfill didn't appear to work; note that with the AlphaImageLoader filter, the image paths are relative to the HTML page, not the CSS, so those declarations aren't working either.)

— Reply to this email directly or view it on GitHub.

jefferyto commented 10 years ago

It should work in IE7, though if it's not working for you in IE8, I doubt it's working in IE7 either.