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

Terminal links - chrome devtools link isn't clickable #21668

Closed
roblourens opened this issue Mar 1, 2017 · 8 comments
Closed

Terminal links - chrome devtools link isn't clickable #21668

roblourens opened this issue Mar 1, 2017 · 8 comments
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s) feature-request Request for new features or functionality help wanted Issues identified as good community contribution opportunities terminal Integrated terminal issues upstream Issue identified as 'upstream' component related (exists outside of VS Code)
Milestone

Comments

@roblourens
Copy link
Member

roblourens commented Mar 1, 2017

These links look like chrome-devtools://devtools/remote/serve_file/@60cd6e859b9f557d2312f5bf532f6aec5f284980/inspector.html?experiments=true&v8only=true&ws=localhost:9229/79e1ddbc-e79f-48c0-953d-60482c6ae5e7 and are only underlined from the first slashes to the first &. Not sure if the problem is the chrome-devtools protocol, or @ character, or something else

@Tyriar
Copy link
Member

Tyriar commented Mar 2, 2017

@roblourens they should no longer be highlighted from tomorrow's build as links are validated now. Does support for such a link make sense? Would telling Chrome to load a link like that work?

@Tyriar Tyriar added terminal Integrated terminal issues info-needed Issue requires more information from poster labels Mar 2, 2017
@roblourens
Copy link
Member Author

I think supporting them makes sense and would be very awesome. To get them to be highlighted, I need to change chrome-devtools to http, and add a .com on devtools, so I guess we would need it to recognize those as valid parts of a url.

@Tyriar
Copy link
Member

Tyriar commented Mar 14, 2017

Is the desired behavior to just open it in the default web browser (and assume it's Chrome?). I'm not sure how reliable google-chrome is on the PATH (does it change with canary, same on all platforms, etc).

@roblourens
Copy link
Member Author

I think that opening with the default browser would be expected (ie treating it like any other link), even though it probably won't work in a non-chrome browser.

@Tyriar
Copy link
Member

Tyriar commented Mar 14, 2017

Upstream issue: xtermjs/xterm.js#599

@Tyriar Tyriar added upstream Issue identified as 'upstream' component related (exists outside of VS Code) and removed info-needed Issue requires more information from poster labels Mar 14, 2017
@aponzani
Copy link

+1 for this to be supported (i.e. opening the link in default browser). FYI, this is the type of debug link produced by running node --inspect --debug-brk mynodefile.js so it would make for a very nice workflow to aid in debugging node scripts from VS Code.

However, FWIW, this link doesn't appear to be clickable in my zsh shell outside of VS Code either.

@Tyriar Tyriar added the help wanted Issues identified as good community contribution opportunities label Apr 24, 2017
@Tyriar Tyriar added this to the Backlog milestone Apr 24, 2017
@Tyriar
Copy link
Member

Tyriar commented Apr 24, 2017

If anyone wants to help with this you can add support here: https://github.com/sourcelair/xterm.js/blob/master/src/Linkifier.ts#L18

Basically you need to allow the new protocol and ensure it's linkified (make sure you can use it without a top level domain; .com, etc.).

@Tyriar Tyriar added the feature-request Request for new features or functionality label Dec 22, 2017
@Tyriar
Copy link
Member

Tyriar commented Jun 21, 2018

Tracking in xtermjs/xterm.js#583

@Tyriar Tyriar closed this as completed Jun 21, 2018
@Tyriar Tyriar added the *duplicate Issue identified as a duplicate of another issue(s) label Jun 21, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Aug 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s) feature-request Request for new features or functionality help wanted Issues identified as good community contribution opportunities terminal Integrated terminal issues upstream Issue identified as 'upstream' component related (exists outside of VS Code)
Projects
None yet
Development

No branches or pull requests

3 participants