-
-
Notifications
You must be signed in to change notification settings - Fork 198
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: Cancel visual mode when switching editor #1578
Conversation
4fa93e6
to
4f1d064
Compare
4f1d064
to
a6cd7ca
Compare
Note I "fixed" a test by changing the assertion. It was changed to |
Off-topic, I think there are too many logs now, one log per step...😂 |
@xiyaowong Yeah. It's pretty crazy. It did help me though with debugging this issue. |
logger.debug( | ||
`Cancel visual mode to prevent selection from previous editor to carry over to active editor`, | ||
); | ||
await this.client.input("<Esc>"); |
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.
:help interrupt()
may be more ideal than inputting ESC.
Fixes #1577