[PR #8921] [CLOSED] Add skeleton code for OSC 7 #27344

Open
opened 2026-01-31 09:21:26 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/8921
Author: @skyline75489
Created: 1/28/2021
Status: Closed

Base: mainHead: chesterliu/dev/osc-7-2021-edition


📝 Commits (7)

  • c435f90 Add skeleton code for OSC 7
  • 54ec9bb Spell check
  • 398e6cb Merge branch 'main' into chesterliu/dev/osc-7-2021-edition
  • 4b9b417 Feedback
  • 44d14b7 Fix unescape
  • a90da70 Merge branch 'main' into chesterliu/dev/osc-7-2021-edition
  • 5ff9e35 Merge branch 'main' into chesterliu/dev/osc-7-2021-edition

📊 Changes

12 files changed (+186 additions, -0 deletions)

View changed files

📝 .github/actions/spelling/expect/expect.txt (+1 -0)
📝 .github/actions/spelling/patterns/patterns.txt (+1 -0)
📝 src/cascadia/TerminalCore/TerminalApi.cpp (+6 -0)
📝 src/cascadia/TerminalCore/TerminalDispatch.cpp (+12 -0)
📝 src/cascadia/TerminalCore/TerminalDispatch.hpp (+2 -0)
📝 src/terminal/adapter/ITermDispatch.hpp (+2 -0)
📝 src/terminal/adapter/adaptDispatch.cpp (+10 -0)
📝 src/terminal/adapter/adaptDispatch.hpp (+2 -0)
📝 src/terminal/adapter/termDispatch.hpp (+2 -0)
📝 src/terminal/parser/OutputStateMachineEngine.cpp (+62 -0)
📝 src/terminal/parser/OutputStateMachineEngine.hpp (+5 -0)
📝 src/terminal/parser/ut_parser/OutputEngineTest.cpp (+81 -0)

📄 Description

Summary of the Pull Request

This adds the skeleton code for OSC 7 to the Windows Terminal. No actual functionality is implemented yet,

References

Related: #3158
Supersedes: #7668
Supports: #8214

PR Checklist

  • Closes #xxx
  • CLA signed. If not, go over here and sign the CLA
  • Tests added/passed
  • Documentation updated. If checked, please file a pull request on our docs repo and link it here: #xxx
  • Schema updated.
  • I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/8921 **Author:** [@skyline75489](https://github.com/skyline75489) **Created:** 1/28/2021 **Status:** ❌ Closed **Base:** `main` ← **Head:** `chesterliu/dev/osc-7-2021-edition` --- ### 📝 Commits (7) - [`c435f90`](https://github.com/microsoft/terminal/commit/c435f90b4666678e6151a5878c2aafa09d02b1cd) Add skeleton code for OSC 7 - [`54ec9bb`](https://github.com/microsoft/terminal/commit/54ec9bbb9cfb22bc501f8e2e8bae2878a875f2e9) Spell check - [`398e6cb`](https://github.com/microsoft/terminal/commit/398e6cb441242ab98a15cc9e536721ac3642f12b) Merge branch 'main' into chesterliu/dev/osc-7-2021-edition - [`4b9b417`](https://github.com/microsoft/terminal/commit/4b9b417546a48ee30007cddc91a5a64871548913) Feedback - [`44d14b7`](https://github.com/microsoft/terminal/commit/44d14b7e81847f4f0d8ed689ca2664e649f39717) Fix unescape - [`a90da70`](https://github.com/microsoft/terminal/commit/a90da702c05b5bb4991da177a1efefdeb2440f5a) Merge branch 'main' into chesterliu/dev/osc-7-2021-edition - [`5ff9e35`](https://github.com/microsoft/terminal/commit/5ff9e355076442739d4dbc0842d69d5e3f10e797) Merge branch 'main' into chesterliu/dev/osc-7-2021-edition ### 📊 Changes **12 files changed** (+186 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `.github/actions/spelling/expect/expect.txt` (+1 -0) 📝 `.github/actions/spelling/patterns/patterns.txt` (+1 -0) 📝 `src/cascadia/TerminalCore/TerminalApi.cpp` (+6 -0) 📝 `src/cascadia/TerminalCore/TerminalDispatch.cpp` (+12 -0) 📝 `src/cascadia/TerminalCore/TerminalDispatch.hpp` (+2 -0) 📝 `src/terminal/adapter/ITermDispatch.hpp` (+2 -0) 📝 `src/terminal/adapter/adaptDispatch.cpp` (+10 -0) 📝 `src/terminal/adapter/adaptDispatch.hpp` (+2 -0) 📝 `src/terminal/adapter/termDispatch.hpp` (+2 -0) 📝 `src/terminal/parser/OutputStateMachineEngine.cpp` (+62 -0) 📝 `src/terminal/parser/OutputStateMachineEngine.hpp` (+5 -0) 📝 `src/terminal/parser/ut_parser/OutputEngineTest.cpp` (+81 -0) </details> ### 📄 Description <!-- Enter a brief description/summary of your PR here. What does it fix/what does it change/how was it tested (even manually, if necessary)? --> ## Summary of the Pull Request This adds the skeleton code for OSC 7 to the Windows Terminal. No actual functionality is implemented yet, <!-- Other than the issue solved, is this relevant to any other issues/existing PRs? --> ## References Related: #3158 Supersedes: #7668 Supports: #8214 <!-- Please review the items on the PR checklist before submitting--> ## PR Checklist * [ ] Closes #xxx * [ ] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [ ] Tests added/passed * [ ] Documentation updated. If checked, please file a pull request on [our docs repo](https://github.com/MicrosoftDocs/terminal) and link it here: #xxx * [ ] Schema updated. * [ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx <!-- Provide a more detailed description of the PR, other things fixed or any additional comments/features here --> ## Detailed Description of the Pull Request / Additional comments <!-- Describe how you validated the behavior. Add automated tests wherever possible, but list manual validation steps taken as well --> ## Validation Steps Performed --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:21:26 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#27344