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

Add an option to pass through input unmodified for WSL mode #89

Closed
rprichard opened this issue Aug 12, 2016 · 4 comments
Closed

Add an option to pass through input unmodified for WSL mode #89

rprichard opened this issue Aug 12, 2016 · 4 comments

Comments

@rprichard
Copy link
Owner

See #82 (comment).

@mintty
Copy link

mintty commented Aug 12, 2016

Thanks a lot for handling this issue.
Passing --mouse to winpty would not be a problem in a preconfigured WSL package to solve mintty/mintty#558, but I wonder whether it would be needed because once escape sequences pass through, mouse sequences would be handled alike; we'll see.
[back in 10 days]

@rprichard
Copy link
Owner Author

The input escape sequences (e.g. mouse movements, button presses/releases) would go through fine, but the output sequences (e.g. enable/disable mouse mode) still wouldn't get through, so mintty would presumably handle mouse input by selecting text, rather than sending mouse escapes to the terminal program.

The --mouse option instructs winpty to output escape sequences to enable mouse mode. Ultimately, it forces winpty to output the escapes here.

Maybe --mouse could work for a preconfigured WSL package, but I think it'd require users to decide between mouse-mode and text-selection-mode before starting a WSL program.

@rprichard
Copy link
Owner Author

I think this approach won't be able to handle the DECCKM mode, where the arrow keys generate an \eO prefix rather than \e[. Because winpty is scraping the output buffer, it wouldn't propagate the \e[?1h string that switches the mode, so the terminal would continue to send normal input. So far, I know that this breaks Midnight Commander (#90).

@mintty
Copy link

mintty commented Sep 16, 2016

This request is now satisfied by wslbridge (many thanks),
so I think this issue can be closed.

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

No branches or pull requests

2 participants