-
Notifications
You must be signed in to change notification settings - Fork 757
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ie7 bugs #17
Comments
Interesting, I definitely would have thought |
If I have time this weekend I'll try and submit a pull request. |
i changed 'in' to '_in', it works on IE8 and 7 |
This seems to still be an issue. I tested it in IE8. Wasn't working. To get around it, I changed the defaults in jquery.textillate.js to my custom options. This prevented me from having to call 'in' in the jquery block of my document. |
Having same issue in IE8 with "in". Any chances next release coming soon? |
Chiming in (!) here to say you can change in to _in as suggested. In your call code, reference _in: and in the textillate.js itself, go to line 149 (for me) and see: if (isInEffect(options.in.effect)) { Yes, change .in. to ._in. there. Two lines down, do it again. And finally, As well. Very quick change. |
I am not asking for support for the animations, but the library itself causes a js error when loaded in compatibility mode, ie7 standards in IE10. The bulk of it seems its from using
.in
as an object property since it is a reserved javascript keyword. Even if i change the.in
to['in']
, http://code.jquery.com/jquery-1.9.1.js seems to have an issue loading at line 2582 sayingmember not found
. Would be great to clean up the code and address these issues to at least make it load in older browsers. Thanks.The text was updated successfully, but these errors were encountered: