-
Notifications
You must be signed in to change notification settings - Fork 119
Support a UI picker of all available targets #19
Comments
Not that useful without nodejs/node#8464, so I'm going to postpone it. Also see https://bugs.chromium.org/p/chromium/issues/detail?id=649823# |
We should implement this feature for existing running processes, so Once Node/Electron/Chrome lands the ability to enable debugging on-the-fly we can expand the usefulness of this feature. |
@roblourens I think this has landed in Node master, as the old V8 protocol has been deprecated. It would be good to get a prototype going. I imagine a flow like:
This would made the attach scenario much simpler for Chrome, and enable a good experience for Node |
Cool, I'll check it out. I'll have to check the version of node that's running before displaying it, hopefully that won't lead to a confusing experience. |
https://bugs.chromium.org/p/chromium/issues/detail?id=649823# was closed and Chrome will not support dynamically entering debug mode anytime soon :'(. I'll continue to use Microsoft/vscode-chrome-debug#280 to track the chrome-specific task of showing a picker of tabs, and microsoft/vscode#25036 to track the node-specific task of entering debug mode dynamically and showing a list of processes. |
See microsoft/vscode#6569
The text was updated successfully, but these errors were encountered: