[PR #5216] [MERGED] Rename copy keybinding arg #26188

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/5216
Author: @carlos-zamora
Created: 4/2/2020
Status: Merged
Merged: 4/3/2020
Merged by: @undefined

Base: masterHead: dev/cazamor/settings-cleanup


📝 Commits (3)

  • f796b48 rename/configure copy kb arg
  • 4e5b45d update docs and schema
  • 7cb6009 Make SettingsSchema.md sound better

📊 Changes

9 files changed (+30 additions, -31 deletions)

View changed files

📝 doc/cascadia/SettingsSchema.md (+1 -1)
📝 doc/cascadia/profiles.schema.json (+3 -3)
📝 src/cascadia/LocalTests_TerminalApp/KeyBindingsTests.cpp (+9 -9)
📝 src/cascadia/TerminalApp/ActionArgs.h (+5 -5)
📝 src/cascadia/TerminalApp/ActionArgs.idl (+1 -1)
📝 src/cascadia/TerminalApp/AppActionHandlers.cpp (+1 -1)
📝 src/cascadia/TerminalApp/TerminalPage.cpp (+3 -4)
📝 src/cascadia/TerminalControl/TermControl.cpp (+3 -3)
📝 src/cascadia/TerminalCore/TerminalSelection.cpp (+4 -4)

📄 Description

Summary of the Pull Request

TrimWhitespace is misleading. So it is now renamed as 'singleLine'. If true, it comes out as a single line! That makes more sense!

PR Checklist

Validation Steps Performed

Attempted the keybinding with both settings (and none set).
Attempted mouse copy with and without shift.


🔄 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/5216 **Author:** [@carlos-zamora](https://github.com/carlos-zamora) **Created:** 4/2/2020 **Status:** ✅ Merged **Merged:** 4/3/2020 **Merged by:** [@undefined](undefined) **Base:** `master` ← **Head:** `dev/cazamor/settings-cleanup` --- ### 📝 Commits (3) - [`f796b48`](https://github.com/microsoft/terminal/commit/f796b484585600f7816e56470090303b8befbe3d) rename/configure copy kb arg - [`4e5b45d`](https://github.com/microsoft/terminal/commit/4e5b45dbdb1c54278b009d8e4fad55e092218462) update docs and schema - [`7cb6009`](https://github.com/microsoft/terminal/commit/7cb6009de53d3a3f8badbcfd650b65ff84b76f69) Make SettingsSchema.md sound better ### 📊 Changes **9 files changed** (+30 additions, -31 deletions) <details> <summary>View changed files</summary> 📝 `doc/cascadia/SettingsSchema.md` (+1 -1) 📝 `doc/cascadia/profiles.schema.json` (+3 -3) 📝 `src/cascadia/LocalTests_TerminalApp/KeyBindingsTests.cpp` (+9 -9) 📝 `src/cascadia/TerminalApp/ActionArgs.h` (+5 -5) 📝 `src/cascadia/TerminalApp/ActionArgs.idl` (+1 -1) 📝 `src/cascadia/TerminalApp/AppActionHandlers.cpp` (+1 -1) 📝 `src/cascadia/TerminalApp/TerminalPage.cpp` (+3 -4) 📝 `src/cascadia/TerminalControl/TermControl.cpp` (+3 -3) 📝 `src/cascadia/TerminalCore/TerminalSelection.cpp` (+4 -4) </details> ### 📄 Description ## Summary of the Pull Request `TrimWhitespace` is misleading. So it is now renamed as 'singleLine'. If true, it comes out as a single line! That makes more sense! ## PR Checklist * [X] Closes #3824 ## Validation Steps Performed Attempted the keybinding with both settings (and none set). Attempted mouse copy with and without shift. --- <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:14:30 +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#26188