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.

Sign up for GitHub

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

Jump to bottom

Clarify 633 E sequence #157635

Merged
merged 2 commits into from
Aug 9, 2022
Merged

Clarify 633 E sequence #157635

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
0817b96
Clarify 633 E sequence
Tyriar Aug 9, 2022
47b3521
Update src/vs/platform/terminal/common/xterm/shellIntegrationAddon.ts
Tyriar Aug 9, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions src/vs/platform/terminal/common/xterm/shellIntegrationAddon.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,9 +92,10 @@ const enum VSCodeOscPt {
* with parsing out the command, such as conpty not guaranteeing the position of the sequence or
* the shell not guaranteeing that the entire command is even visible.
*
* The command line should escape ascii characters using the `\xAB` format, where AB are the
* The command line can escape ascii characters using the `\xAB` format, where AB are the
* hexadecimal representation of the character code (case insensitive), and escape the `\`
* character using `\\`. This is particularly important for new line and semi-colon.
* character using `\\`. It's required to escape semi-colon (`0x3b`) and characters 0x20 and
* below, this is particularly important for new line and semi-colon.
*
* Some examples:
*
Expand Down