Skip to content
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

Fixed issue #478 #479

Closed
wants to merge 1 commit into from
Closed

Conversation

coodoo
Copy link

@coodoo coodoo commented Dec 1, 2014

Confirmed working on osx 10.10 with latest Chrome canary.

@3y3
Copy link
Member

3y3 commented Dec 1, 2014

Landed as 326b05e

Thank you for contribution.

@3y3 3y3 closed this Dec 1, 2014
@coodoo coodoo deleted the front-end-fix branch December 1, 2014 22:07
@chevcast
Copy link

chevcast commented Dec 2, 2014

Can we get node-inspector updated on npm with this fix? I currently cannot expand any nodes in variable hover overs, watch expressions, or property nodes after typing an object reference into the console. Without being able to expand watch expressions I pretty much can't use node-inspector.

@coodoo
Copy link
Author

coodoo commented Dec 3, 2014

@chevex I know that's such a total pain right? 😉

For a quick fix, you could replace your local treeoutline.js with the latest one from the repo. On mac it should be located under /usr/local/lib/node_modules

@chevcast
Copy link

chevcast commented Dec 3, 2014

For a quick fix I just installed this repository globally.

npm install -g node-inspector/node-inspector

dpolivy added a commit to dpolivy/iisnode that referenced this pull request Mar 27, 2017
The version of node-inspector being used is very old, and contains some
calls which no longer work in Chrome. node-inspector itself fixed this
bug in December of 2014, and it's frustrating enough for me that I
ported the changes into iisnode's version of the debugger.

Original fix: node-inspector/node-inspector#479
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants