[PR #15198] [MERGED] Added Close Pane to Context Menu #30491

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/15198
Author: @joadoumie
Created: 4/18/2023
Status: Merged
Merged: 4/26/2023
Merged by: @microsoft-github-policy-service[bot]

Base: mainHead: dev/joadoumie/add_close_pane_to_context_menu


📝 Commits (4)

  • b3f78dc Added Close Pane to Context Menu
  • b5f1753 Ran code formatter
  • 722da1f Removed icon and moved leaf count pane check to _UpdateActivePane()
  • 86e3bf6 Update src/cascadia/TerminalApp/TerminalTab.cpp

📊 Changes

3 files changed (+37 additions, -4 deletions)

View changed files

📝 src/cascadia/TerminalApp/Resources/en-US/Resources.resw (+7 -4)
📝 src/cascadia/TerminalApp/TerminalTab.cpp (+28 -0)
📝 src/cascadia/TerminalApp/TerminalTab.h (+2 -0)

📄 Description

Summary of the Pull Request

Adding a 'Close Pane' menu item in the context menu.

References and Relevant Issues

#13580

Detailed Description of the Pull Request / Additional comments

If a user decides to split a tab to create multiple panes through the context menu, they should be able to then close the pane via the context menu too. This PR introduces a new context menu item, 'Close Pane', that only appears when a user has 2 or more panes in a tab. When a user clicks close pane, the _active_pane will be closed.

Validation Steps Performed

close_pane_terminal

As it's my first PR, I still need to understand how to go through the testing suite.

PR Checklist


🔄 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/15198 **Author:** [@joadoumie](https://github.com/joadoumie) **Created:** 4/18/2023 **Status:** ✅ Merged **Merged:** 4/26/2023 **Merged by:** [@microsoft-github-policy-service[bot]](https://github.com/apps/microsoft-github-policy-service) **Base:** `main` ← **Head:** `dev/joadoumie/add_close_pane_to_context_menu` --- ### 📝 Commits (4) - [`b3f78dc`](https://github.com/microsoft/terminal/commit/b3f78dc8daefe76d8b7d3990db53e2b446735b2f) Added Close Pane to Context Menu - [`b5f1753`](https://github.com/microsoft/terminal/commit/b5f1753b8f887dad24bbcced9ea7e0e665f482fd) Ran code formatter - [`722da1f`](https://github.com/microsoft/terminal/commit/722da1f3306d9e879f8f20c08244d7ca27a33576) Removed icon and moved leaf count pane check to _UpdateActivePane() - [`86e3bf6`](https://github.com/microsoft/terminal/commit/86e3bf6237753f88b17050271909f11cbdd469e1) Update src/cascadia/TerminalApp/TerminalTab.cpp ### 📊 Changes **3 files changed** (+37 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `src/cascadia/TerminalApp/Resources/en-US/Resources.resw` (+7 -4) 📝 `src/cascadia/TerminalApp/TerminalTab.cpp` (+28 -0) 📝 `src/cascadia/TerminalApp/TerminalTab.h` (+2 -0) </details> ### 📄 Description ## Summary of the Pull Request Adding a 'Close Pane' menu item in the context menu. ## References and Relevant Issues #13580 ## Detailed Description of the Pull Request / Additional comments If a user decides to split a tab to create multiple panes through the context menu, they should be able to then close the pane via the context menu too. This PR introduces a new context menu item, 'Close Pane', that only appears when a user has 2 or more panes in a tab. When a user clicks close pane, the _active_pane will be closed. ## Validation Steps Performed ![close_pane_terminal](https://user-images.githubusercontent.com/98557455/232649000-8b521070-4f1b-4da9-8092-6ff802e91e2c.gif) As it's my first PR, I still need to understand how to go through the testing suite. ## PR Checklist - [x] Closes #13580 - [ ] 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 (if necessary) --- <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:41:09 +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#30491