-
Notifications
You must be signed in to change notification settings - Fork 35
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
Doesn't work with jQuery UI 1.9.0+ #11
Comments
Hi Mike, Thanks for looking into this. I need all the help I can get to keep this maintained;-) To be honest, I am moving away from jQuery UI wherever possible and not using on new projects. So I'm wondering if rather than make it jQuery UI 1.9 compatible, would it be best to refactor this completely to remove the jQueryUI dependency? |
I haven't done much plugin development with jQuery UI so I'll go with whatever is your suggestion. |
Removing dependency on jQuery UI is a good idea, because the download is very large (can be 200 kbytes plus). Currently trying to use this and falling foul of this problem - is there a known workaround? |
I made this work wtih 1.9 by making the following changes in jquery.addtocal.js:
|
@cincobyte good stuff. If you can send a pull request with your changes, I'm happy to roll them in. |
Hi, |
When I updated jQuery UI to version 1.9.0 or higher I start getting the following error:
cannot call methods on AddToCal prior to initialization; attempted to call method 'close'
This stems from the following call within toggleMenu:
$('.ui-addtocal').AddToCal( 'close' );
The text was updated successfully, but these errors were encountered: