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
Make file paths with spaces in the console work as clickable links #1995
Comments
I'm not sure this is something the extension controls. It is the VSCode terminal that turns paths into clickable links (not this extension). And the extension doesn't control the generation of that path and the fact that it isn't quoted. It is PowerShell that generates that error text. |
Oops, had a comment written out but forgot to post and @rkeithhill's since commented! @PrzemyslawKlys I'm not sure how we'd change this; these links appear because VSCode automatically turns paths into links (i.e. we don't do anything, it's a native VSCode feature). If you get rid of the PowerShell extension and replicate the same error in the Without us intercepting and reformatting the error message in the integrated console (which I imagine might be unwanted by many users and would come at a performance and complexity cost), I'm not sure what we can do here |
I figured it might not be something straight forward. If it can't be fixed I can close this issue. It's not a big deal, just something that would be useful in the long run. Maybe a fix to PS6/PS7 so that throwing errors would include path in quotes? Still, older versions of PS would have the problem, but at least new releases would work. |
Good thinking! |
We should investigate how VSCode does this first -- it may be that quoting won't help |
@Tyriar any thoughts here? |
Quoting doesn't seem to work. :-( |
Yeah I had a play around with it and I suspect the link support starts after the quote, where the |
System Details
System Details Output
Issue Description
When something is broken, and an error is thrown VSCode allows you to ctrl+click link to link to file. This is great, but it doesn't work with the path that has spaces in them.
Expected Behaviour
Should work with paths with spaces.
The text was updated successfully, but these errors were encountered: