[PR #18486] Bugfix: only round to the nearest cell for selection #31544

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

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

State: closed
Merged: Yes


Summary of the Pull Request

Fixes a bug where VT mouse mode would round to the nearest cell when clicking the mouse button.
The fix is to round to the nearest cell only when we're selecting text. The other scenarios affected are:

  • clicking on a hyperlink
  • vt mouse mode
  • where the context menu is anchored

Really the most notable ones were the first two. So now, we use the position of the cell we clicked on. We only round for selection.

References and Relevant Issues

Follow-up to #18106

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

Opened Midnight Commander in Ubuntu and clicked between the two panes.

  • Before: threshold was too early to switch between panes
  • After: threshold is clearly separated between the outline of the two panes
**Original Pull Request:** https://github.com/microsoft/terminal/pull/18486 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request Fixes a bug where VT mouse mode would round to the nearest cell when clicking the mouse button. The fix is to round to the nearest cell only when we're selecting text. The other scenarios affected are: - clicking on a hyperlink - vt mouse mode - where the context menu is anchored Really the most notable ones were the first two. So now, we use the position of the cell we clicked on. We only round for selection. ## References and Relevant Issues Follow-up to #18106 ## Detailed Description of the Pull Request / Additional comments ## Validation Steps Performed Opened Midnight Commander in Ubuntu and clicked between the two panes. - Before: threshold was too early to switch between panes - After: threshold is clearly separated between the outline of the two panes
claunia added the pull-request label 2026-01-31 09:47:56 +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#31544