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

Size specifications, IE7 fix and dynamic styles #8

Closed eric1234 closed 11 years ago

eric1234 commented 11 years ago

When using this polyfill I found the following issues:

This is a rough patch. I basically just fixed it up for my website, copy and pasted the final code into GitHub and submitted the patch in hopes it is useful to someone else. If I get a chance down the road I can try to make a proper patch (commit for each issue I am fixing and more extensive testing to make sure I didn't break anything). But I figured a messy patch is better than no patch if this helps someone out and I never get a chance to clean it up.

jefferyto commented 11 years ago

Thanks for this patch! I wasn't able to use you code as-is while developing 0.2.0 but your ideas are definitely in there :-)