[PR #2762] [MERGED] Set actual background color in HTML copy #25058

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/2762
Author: @mcpiroman
Created: 9/15/2019
Status: Merged
Merged: 9/16/2019
Merged by: @carlos-zamora

Base: masterHead: html-bg-color


📝 Commits (3)

📊 Changes

4 files changed (+11 additions, -7 deletions)

View changed files

📝 src/buffer/out/textBuffer.cpp (+3 -5)
📝 src/buffer/out/textBuffer.hpp (+1 -0)
📝 src/cascadia/TerminalControl/TermControl.cpp (+5 -1)
📝 src/interactivity/win32/Clipboard.cpp (+2 -1)

📄 Description

Summary of the Pull Request

It just takes background color from terminal (or console) configured in settings, instead first copied
character. Somewhat easier than I thought it would be.

PR Checklist

  • Closes cmd: Pressing tilde inserts unexpected character (#2128)
  • CLA signed. If not, go over here and sign the CLA
  • Tests added/passed
  • Requires documentation to be 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: #990

🔄 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/2762 **Author:** [@mcpiroman](https://github.com/mcpiroman) **Created:** 9/15/2019 **Status:** ✅ Merged **Merged:** 9/16/2019 **Merged by:** [@carlos-zamora](https://github.com/carlos-zamora) **Base:** `master` ← **Head:** `html-bg-color` --- ### 📝 Commits (3) - [`55d319f`](https://github.com/microsoft/terminal/commit/55d319f311d1ecb5f8c290cc4b26e6478f745c86) Set actual background color in HTML copy - [`0036468`](https://github.com/microsoft/terminal/commit/0036468724841dfb41777e6321a3890d9cff0652) const - [`4fc59f0`](https://github.com/microsoft/terminal/commit/4fc59f0a621d758740fc9b9ca6e471f3eb214d23) format ### 📊 Changes **4 files changed** (+11 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `src/buffer/out/textBuffer.cpp` (+3 -5) 📝 `src/buffer/out/textBuffer.hpp` (+1 -0) 📝 `src/cascadia/TerminalControl/TermControl.cpp` (+5 -1) 📝 `src/interactivity/win32/Clipboard.cpp` (+2 -1) </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 It just takes background color from terminal (or console) configured in settings, instead first copied character. Somewhat easier than I thought it would be. <!-- Please review the items on the PR checklist before submitting--> ## PR Checklist * [X] Closes #2128 * [X] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [ ] Tests added/passed * [ ] Requires documentation to be updated * [X] 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: #990 --- <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:06:59 +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#25058