-
Notifications
You must be signed in to change notification settings - Fork 67
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
jQuery doesn't work on node 0.10.0 #52
Comments
Getting the same error.
|
Same error here... old: if(window == null ) {
window = require('jsdom').jsdom().createWindow(); new: if(!window || !window.document) {
window = require('jsdom').createWindow();
window.document = require('jsdom').jsdom(); (maybe need to upgrade the jsdom and xmlhttprequest packeges too) |
Having the same problem |
Me too! Any idea? |
I believe the problem is actually to do with the underlying jsdom library, but recent updates have fixed it. Specifically, this commit: Can someone publish a new version of the package that has this commit? |
@NeverI I did the changes you mentioned and it worked, thanks. For the record, I tried with node v0.10 and v0.8, reinstalled a few times.. upgrading the jsdom version ( as @gorillamania ) didn't changed anything. |
Just so everyone knows the state and future of the npm jQuery package, please see this Issue #59 |
The comments provided by Never! worked. Upgrading the jsdom and xmlhttprequest didn't help me. |
NeverI's hack worked for me too. |
Never's hack works fine. Although i am a bit puzzled how at one time everything worked well and after re-boot and before the hack, it didn't work. But the hack did work. |
@netroy, would you be willing to fork this project with your changes and get it up on npm? |
@johnaschroeder https://npmjs.org/package/jquery-loader |
Perfect! Thanks for making that available. |
Note from repo owner: Just so everyone knows the state and future of the npm jQuery package, please see this Issue #59
Is it possible that jQuery's not working on node 0.10.0?
I'm getting this error:
The text was updated successfully, but these errors were encountered: