[PR #4404] selection: Shift-Click to set endpoint, clear & discard keypress on ESC #25751

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

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

State: closed
Merged: Yes


This PR addresses the following two issues:

#4203: If a selection is active, a shift-LeftClick will set the SelectionEnd to where the pointer is.
#3911: Currently, any keypress will clear selection, and will pass through to the terminal. This PR will make it so that if a selection is active, any keypress will clear the selection and then any keypress except esc will pass through to the terminal.

PR Checklist

Validation Steps Performed

Played around a whole bunch with shift-clicking selections and regular clicking selections.
Also played around with selections and dismissing with all sorts of keypresses and keychords.
Tests all pass still!

**Original Pull Request:** https://github.com/microsoft/terminal/pull/4404 **State:** closed **Merged:** Yes --- This PR addresses the following two issues: #4203: If a selection is active, a <kbd>shift</kbd>-LeftClick will set the SelectionEnd to where the pointer is. #3911: Currently, any keypress will clear selection, and will pass through to the terminal. This PR will make it so that if a selection is active, _any_ keypress will clear the selection and then any keypress _except_ <kbd>esc</kbd> will pass through to the terminal. ## PR Checklist * [x] Closes #4203; Closes #3911 * [x] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [x] Tests added/passed ## Validation Steps Performed Played around a whole bunch with shift-clicking selections and regular clicking selections. Also played around with selections and dismissing with all sorts of keypresses and keychords. Tests all pass still!
claunia added the pull-request label 2026-01-31 09:11:33 +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#25751