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

Incorrect results for Windows Terminal #7

Closed
dropsonic opened this issue Nov 20, 2020 · 8 comments · Fixed by #8
Closed

Incorrect results for Windows Terminal #7

dropsonic opened this issue Nov 20, 2020 · 8 comments · Fixed by #8

Comments

@dropsonic
Copy link
Contributor

dropsonic commented Nov 20, 2020

Starting with v1.4, Windows Terminal supports embedded hyperlinks.
However, supports-hyperlinks returns false for this terminal.

As a workaround, I set the environment variable FORCE_HYPERLINK, and everything started working fine.

It happens because there is a piece of code that disables hyperlink support everywhere on Windows (which is not correct):

if (process.platform === 'win32') {
    return false;
}

It is possible to understand that we're running in Windows Terminal by checking env.WT_SESSION existence; however, it is not so easy to get the actual version of the terminal. See microsoft/terminal#1040 for more details.

@jamestalmage
Copy link
Owner

PR Welcome.

@dropsonic
Copy link
Contributor Author

@jamestalmage it is not possible to properly implement it until microsoft/terminal#1040 is completed.

@abhijit-hota
Copy link

abhijit-hota commented Dec 7, 2020

I was finding ways to know the version number as you said.

One way I thought of was to execute wt -v (wt is included in the Path!).
But it opened up an alert window rather than printing it.

I asked why and they said the reason

image

@Richienb
Copy link

Richienb commented Jan 8, 2021

One way I thought of was to execute wt -v (wt is included in the Path!).

image

@Richienb
Copy link

Richienb commented Jan 8, 2021

Right now, this is the best way I could find to do it.

@abhijit-hota
Copy link

abhijit-hota commented Jan 8, 2021 via email

@Richienb
Copy link

@abhijit-hota Perhaps we can get the version from the Windows Store package.

@dropsonic
Copy link
Contributor Author

According to Microsoft, there is no need to check the exact version of the terminal.
I've created a PR: #8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants