-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Fix win key resets scroll location #2514
Fix win key resets scroll location #2514
Conversation
Thanks for doing this! It might be better for us to handle it up in TermControl's PreviewKeyDown handler -- @zadjii-msft, what do you think? |
@DHowett-MSFT this seems reasonable to me. I will change the logic |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is excellent, thanks!
Thanks for the contribution! |
🎉 Handy links: |
Summary of the Pull Request
Checking if a windows key is pressed and prevent handling at all.
PR Checklist
Detailed Description of the Pull Request / Additional comments
Validation Steps Performed
The terminal should not reset the scroll location
This is my very first pull request on GitHub. Please be kind 😃