[PR #14636] Make sure to update the selection pivot while circling #30179

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

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

State: closed
Merged: Yes


This builds upon #10749. When we added a separate pivot to track the "active" anchor, we forgot to update the pivot while circling. What does that mean? Moving the mouse would trigger us to update the selection using new endpoint and the old pivot, which hadn't been updated.

There's probably a more elegant way of doing this, but it's good enough.

Updated the test to cover this as well.

Closes #14462

**Original Pull Request:** https://github.com/microsoft/terminal/pull/14636 **State:** closed **Merged:** Yes --- This builds upon #10749. When we added a separate pivot to track the "active" anchor, we forgot to update the pivot while circling. What does that mean? Moving the mouse would trigger us to update the selection using new endpoint and the old _pivot_, which hadn't been updated. There's probably a more elegant way of doing this, but it's good enough. Updated the test to cover this as well. Closes #14462
claunia added the pull-request label 2026-01-31 09:39:08 +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#30179