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
Polish command navigation for "empty commands" for non-partial terminal capabilities #155551
Conversation
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.
Partial command detection is what's used when shell integration is disabled, so this is correct in that it treats them all the same.
src/vs/workbench/contrib/terminal/browser/xterm/commandNavigationAddon.ts
Outdated
Show resolved
Hide resolved
bbdc71d
to
0d3b073
Compare
0d3b073
to
a0e4e29
Compare
Hi @Tyriar, I have improved the behaviour of the skipping empty commands. |
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.
Works well, thanks @MonadChains!
This PR fixes #155543
This PR implements the skipping of empty commands in the case of the terminal with non-partial command detection capabilities. The result is:
However in the case of partial command detection capabilities (i.e. IPartialCommandDetectionCapability ) I was not able to implement it. I don't fully understand when terminals with partial detection capabilities occurs, so, I couldn't trigger and investigate them.