Added IE6+ support. #8
Conversation
|
Actually, it should now work in any relatively recent pre JavaScript 1.6 browser. |
|
I started working on this myself, then realized it was already done. Thanks... Hopefully this will get merged in. |
--- Ive added support for IE 6 and later. Also, I added in semicolons in after function declarations, as is best practice. Just remove them if you prefer it without.
|
Added it, but did some bigger modifications. Was the stuff in addEvent where it added functions on the object to prevent memory leaks? I'm not sure if that would work at all in this situation (and we don't deregister anything), so I've left it out for now. |
|
The addEvent function was actually from a blog post by John Resig. As far as I know, adding the functions to the object was just so they could be removed later. Some of the code was for making sure this was assigned correctly in the callback in IE, but as that's not used in keymaster.js, it's not necessary. |
I've added support for IE 6 and later.
Also, I added in semicolons in after function declarations, as is best practice. Just remove them if you prefer it without.