-
-
Notifications
You must be signed in to change notification settings - Fork 121
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
use CSI 14 t where TIOCGWINSZ ioctl isn't available #1891
Comments
though it might be that neither of these will work with Windows Terminal: microsoft/terminal#8581 |
So in this case, I'd NOT use Windows Terminal as the base. Instead I'd look at the other terminals that run on Windows. Unfortunately though, since ConPTY doesn't have a pass-through mode, you're likely going to find very few terminal emulators capable of supporting this. MinTTY for example would be a terminal that completely supports these along with Sixel and may be your only viable option for testing. If you do choose to use that terminal, you don't need to install Cygwin or Msys2 to run it, the necessary dlls are included in WSLtty or Git bash. Note: Some of the terminals existed prior to ConPTY so it's possible you could use the winpty mode instead to get these sequences working as I'd expect if those terminals support Sixel, they'd support the CSI 14 t sequence. If they don't....I'd raise an issue with them to do so when in winpty mode vice ConPTY mode with the expectation of moving that logic to work in ConPTY "Passthrough" Mode when it becomes available. |
thinking we might want to throw this into the initial queries, so we'll know when it's available? |
we're now sending CSI 14t in our initial query, and parsing the response. confirmed that it works. |
on at least windows, i assume we don't have
TIOCGWINSZ
as anioctl
(maybe we do? who knows). when we move to support it, try usingCSI 14 t
to recover the window geometry in pixels. to use this as a substitute, we would need put it (and the resultant round trip) intonotcurses_resize()
, which is kinda lame, so look for a native call that includes this functionality.The text was updated successfully, but these errors were encountered: