[PR #1302] Multi-Click Selection: Triple-Click Settings + Viewport Selection #24575

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

Original Pull Request: https://github.com/microsoft/terminal/pull/1302

State: closed
Merged: No


Summary of the Pull Request

Introduces "viewport selection" which creates a selection of the entire viewport. Also, adds per-profile settings to decide whether a triple-click performs a line selection or viewport selection (disabling it is also an option).

References

Builds on #1197. Similar to #1273.
Closes #1084.

PR Checklist

  • Closes Move hard-coded strings and UI text to Resources (#1084)
  • 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.

Detailed Description of the Pull Request / Additional comments

  • TerminalSettings:
    • add property to winrt TerminalSettings
  • Terminal App:
    • read/write JSON for this new property
    • default set to line selection
  • TerminalCore:
    • import/use selection mode from ICoreSettings
    • perform proper action
  • UnitTests_TerminalCore:
    • Added setting to MockTermSettings

Validation Steps Performed

  • modified profiles.json. Then performed triple click in an instance of that profile
**Original Pull Request:** https://github.com/microsoft/terminal/pull/1302 **State:** closed **Merged:** No --- <!-- 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 Introduces "viewport selection" which creates a selection of the entire viewport. Also, adds per-profile settings to decide whether a triple-click performs a line selection or viewport selection (disabling it is also an option). <!-- Other than the issue solved, is this relevant to any other issues/existing PRs? --> ## References Builds on #1197. Similar to #1273. Closes #1084. <!-- Please review the items on the PR checklist before submitting--> ## PR Checklist * [ ] Closes #1084 * [ ] 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. <!-- 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 - TerminalSettings: - add property to winrt TerminalSettings - Terminal App: - read/write JSON for this new property - default set to line selection - TerminalCore: - import/use selection mode from ICoreSettings - perform proper action - UnitTests_TerminalCore: - Added setting to `MockTermSettings` <!-- Describe how you validated the behavior. Add automated tests wherever possible, but list manual validation steps taken as well --> ## Validation Steps Performed - modified profiles.json. Then performed triple click in an instance of that profile
claunia added the pull-request label 2026-01-31 09:04:06 +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#24575