-
-
Notifications
You must be signed in to change notification settings - Fork 108
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
feature request: detect Windows Terminal #59
Comments
Probably, to check environment variable |
Yes I know that way too |
Currently, I have no best idea to do it. |
Let's keep it open until it is implemented. |
Are you trying to detect Windows Terminal, or whether VT100 is supported? Talking with @DHowett - who works on Windows Terminal - in cli/cli#4106, to better detect or even enable VT100 support (or close to it) which even conhost.exe has supported for a while, you should actually call So maybe a platform-specific implementation of something like |
Yes, I now have to check that if |
16 color indexes were always supported. Enabling virtual processing enables 256color or truecolor depending on the terminal. Windows Terminal supports truecolor. There's also a quirk of the last column being used when VT processing is not enabled, which we're dealing with in cli/cli#4106. |
Oh I meant to write True Color 😅 |
FYI, go-colorable already support ENABLE_VIRTUAL_TERMINAL_PROCESSING. |
I saw that, but it assumes coloring is enabled if it can't get or set the console mode. In the case we needed, that assumption would be false. Besides, it's more than just coloring. In a library such as |
Sorry, I still not understand what go-isatty can do. Adding new function like |
VT100 support is not just in Windows Terminal, but in conhost as of at least Windows 7 as well. I think just adding an If VT100 is supported (at least in part - seems few, if any terminals, support it all), it's not just coloring that is possible, hence why it should be separate from |
This is a code to check DRCS Sixel. UNIX terminal should send-back TerminalID for terminal report. At least, we can check support of VT100. https://github.com/mattn/longcat/blob/62bfdec3ef9bebd028124cfee9f9d5e8d7eab938/main.go#L227 |
Could you please test this? https://gist.github.com/mattn/00cf5b7e38f4cceaf7077f527479870c |
Yes, it displayed a long cat in the following configurations:
|
Hey! I really like your library and I will be using it for my module for detecting 8 bit color support for Windows Console. As this module also supports Cygwin, can you also add detection for Windows Terminal as it is entirely different from Windows Console.
Thanks!
The text was updated successfully, but these errors were encountered: