Skip to content

Use new strategy for ConsoleReadLine.ReadKeyAsync to fix macOS and Linux issues #400

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

Merged
merged 1 commit into from
Mar 22, 2017

Conversation

daviwil
Copy link
Contributor

@daviwil daviwil commented Mar 22, 2017

This change uses a new strategy for providing a quasi-async
Console.ReadKey implementation so that key handling works better on Linux
and macOS. This should eliminate issues where keys and escape sequences
were being echoed incorrectly. It should also eliminate the issue where
the backspace key is unusable on these platforms.

Resolves PowerShell/vscode-powershell#533.
Resolves PowerShell/vscode-powershell#542.

@daviwil daviwil added this to the 0.11.0 milestone Mar 22, 2017
@daviwil daviwil changed the title Use new strategy for ConsoleReadLine.ReadKeyAsync to fix macOS and Li… Use new strategy for ConsoleReadLine.ReadKeyAsync to fix macOS and Linux Mar 22, 2017
@daviwil daviwil changed the title Use new strategy for ConsoleReadLine.ReadKeyAsync to fix macOS and Linux Use new strategy for ConsoleReadLine.ReadKeyAsync to fix macOS and Linux issues Mar 22, 2017
…nux issues

This change uses a new strategy for providing a quasi-async
Console.ReadKey implementation so that key handling works better on Linux
and macOS.  This should eliminate issues where keys and escape sequences
were being echoed incorrectly.  It should also eliminate the issue where
the backspace key is unusable on these platforms.

Resolves PowerShell/vscode-powershell#533.
Resolves PowerShell/vscode-powershell#542.
@daviwil daviwil force-pushed the improved-readline branch from 1f415b9 to 8c1a086 Compare March 22, 2017 00:14
@daviwil daviwil merged commit 8efdcc3 into PowerShell:develop Mar 22, 2017
@daviwil daviwil deleted the improved-readline branch March 22, 2017 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants