[PR #3595] [MERGED] Conhost: copy RTF to clipboard #25442

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/3595
Author: @anirudhrb
Created: 11/15/2019
Status: Merged
Merged: 12/5/2019
Merged by: @undefined

Base: masterHead: rtf_copy_conhost


📝 Commits (3)

  • 9c3838d Conhost: copy RTF to clipboard
  • 8050584 Added description for CopyToSystemClipboard function
  • c5c21ff Renamed vars as per code review suggestions

📊 Changes

3 files changed (+56 additions, -41 deletions)

View changed files

📝 src/interactivity/win32/Clipboard.cpp (+50 -35)
📝 src/interactivity/win32/clipboard.hpp (+4 -4)
📝 src/interactivity/win32/windowio.cpp (+2 -2)

📄 Description

RTF data is now copied to the clipboard. The clipboard format name
used for RTF data is Rich Text Format.

Refactored some code in Clipboard.cpp so that the code for setting
data to the clipboard is re-used. Also, renamed parameter
fAlsoCopyHtml to fAlsoCopyFormatting to make it more generic.

Tested by copying text from console to WordPad. Also verified that
HTML copy is not regressed by copying to Word.

Closes #3560.


🔄 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/3595 **Author:** [@anirudhrb](https://github.com/anirudhrb) **Created:** 11/15/2019 **Status:** ✅ Merged **Merged:** 12/5/2019 **Merged by:** [@undefined](undefined) **Base:** `master` ← **Head:** `rtf_copy_conhost` --- ### 📝 Commits (3) - [`9c3838d`](https://github.com/microsoft/terminal/commit/9c3838daa55178f3d33a09d0d1c97ac2b4ebbca1) Conhost: copy RTF to clipboard - [`8050584`](https://github.com/microsoft/terminal/commit/805058430093cbf71698d61d4140b8213a81a5d7) Added description for CopyToSystemClipboard function - [`c5c21ff`](https://github.com/microsoft/terminal/commit/c5c21ffb0babffd5dfb7e3391d4596ed7a4cfbb2) Renamed vars as per code review suggestions ### 📊 Changes **3 files changed** (+56 additions, -41 deletions) <details> <summary>View changed files</summary> 📝 `src/interactivity/win32/Clipboard.cpp` (+50 -35) 📝 `src/interactivity/win32/clipboard.hpp` (+4 -4) 📝 `src/interactivity/win32/windowio.cpp` (+2 -2) </details> ### 📄 Description RTF data is now copied to the clipboard. The clipboard format name used for RTF data is `Rich Text Format`. Refactored some code in `Clipboard.cpp` so that the code for setting data to the clipboard is re-used. Also, renamed parameter `fAlsoCopyHtml` to `fAlsoCopyFormatting` to make it more generic. Tested by copying text from console to WordPad. Also verified that HTML copy is not regressed by copying to Word. Closes #3560. --- <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:09:32 +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#25442