[PR #6431] Don't snap on input nor dismiss selection for just a modifier key #26685

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

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

State: closed
Merged: Yes


Does what it says on the label. Pure modifier keys weren't making it
this far at all prior to #6309. This PR changes these methods to make
sure that we only dismiss a selection or snap on input when the key
pressed isn't a modifier key.

References

PR Checklist

Validation Steps Performed

  • Tried to repro this in the Terminal, couldn't anymore.
**Original Pull Request:** https://github.com/microsoft/terminal/pull/6431 **State:** closed **Merged:** Yes --- Does what it says on the label. Pure modifier keys weren't making it this far at all prior to #6309. This PR changes these methods to make sure that we only dismiss a selection or snap on input when the key pressed isn't a modifier key. ## References * regressed in #6309 ## PR Checklist * [x] Closes #6423 * [x] I work here * [ ] Tests added/passed * [n/a] Requires documentation to be updated ## Validation Steps Performed * Tried to repro this in the Terminal, couldn't anymore.
claunia added the pull-request label 2026-01-31 09:17: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#26685