IE6, IE7 maybe IE8 #7

Closed
dominicglenn opened this Issue Jul 31, 2012 · 1 comment

Projects

None yet

2 participants

@dominicglenn

This plugin does not work on those versions of IE.

The first issue is that they do not support addEventListener, and require attachEvent to be used instead; after I modified my local copy to account for this I still go errors about this.async on line 37.

This needs looking at for compatibility with those versions of IE.

@d0ugal
Member
d0ugal commented Jul 31, 2012

Thanks for the report.

Can you send a pull request with your modifications to handle addEventListener? I don't have a machine with IE8 or IE7 anymore, but if you can provide a fix and test them I'm more than happy to accept them.

There is a fairly comprehensive test suite, so you can send the browser to http://locachejs.org/tests/index.html to run them.

@d0ugal d0ugal added a commit that closed this issue Aug 19, 2012
@d0ugal d0ugal Some cosmetic changes and fixes for IE.
Fixes #7
6ed17d7
@d0ugal d0ugal closed this in 6ed17d7 Aug 19, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment