Mark mode hyperlink navigation sometimes skips over entries #18313

Open
opened 2026-01-31 06:10:08 +00:00 by claunia · 2 comments
Owner

Originally created by @carlos-zamora on GitHub (Aug 26, 2022).

Originally assigned to: @carlos-zamora on GitHub.

Here's a bunch of bugs we found during our bug bash that are related to tab to hyperlink and mark mode

  • If Shift+tab causes scroll and new selection is on multiple lines, we scroll to show the "end" (should show the whole thing)
  • Shift+tab skips over hyperlink that is (end) partially included in top row of view
  • If a selection exists, mark mode should happen on the existing selection (#13893)
  • Ctrl+enter on random selection clears selection. Maybe it should try to treat selection as a URL?
  • Mark mode --> move to middle of word --> ExpandSelectionToWord --> Shift+Right/Left --> weird updated selection (#13893 )
  • https://github.com/microsoft/terminal/pull/13494#issuecomment-1234559792
  • remove _selectionIsTargetingUrl, instead check if new selection matches old selection. If so, skip it.
Originally created by @carlos-zamora on GitHub (Aug 26, 2022). Originally assigned to: @carlos-zamora on GitHub. Here's a bunch of bugs we found during our bug bash that are related to tab to hyperlink and mark mode - [ ] If Shift+tab causes scroll and new selection is on multiple lines, we scroll to show the "end" (should show the whole thing) - [ ] Shift+tab skips over hyperlink that is (end) partially included in top row of view - [x] If a selection exists, mark mode should happen on the existing selection (#13893) - [x] Ctrl+enter on random selection clears selection. Maybe it should try to treat selection as a URL? - [x] Mark mode --> move to middle of word --> ExpandSelectionToWord --> Shift+Right/Left --> weird updated selection (#13893 ) - [ ] https://github.com/microsoft/terminal/pull/13494#issuecomment-1234559792 - [ ] remove `_selectionIsTargetingUrl`, instead check if new selection matches old selection. If so, skip it.
claunia added the Help WantedIssue-BugArea-InteractionProduct-Terminal labels 2026-01-31 06:10:08 +00:00
Author
Owner

@carlos-zamora commented on GitHub (Sep 1, 2022):

Shift+tab skips over hyperlink that is (end) partially included in top row of view

Urgh. This is a problem with how we're using the pattern interval tree. Consider the following:

www.github.com/microsoft/term
------------------------------------- View Boundary (Top)
inal/issues/13854

The pattern interval tree doesn't detect the partial match. This issue can be replicated by ctrl+clicking.

@carlos-zamora commented on GitHub (Sep 1, 2022): > Shift+tab skips over hyperlink that is (end) partially included in top row of view Urgh. This is a problem with how we're using the pattern interval tree. Consider the following: ``` www.github.com/microsoft/term ------------------------------------- View Boundary (Top) inal/issues/13854 ``` The pattern interval tree doesn't detect the partial match. This issue can be replicated by ctrl+clicking.
Author
Owner

@zadjii-msft commented on GitHub (Dec 1, 2022):

Oh huh, I wonder if the selection changes referenced here also caused #14464 and #14462 🤔

@zadjii-msft commented on GitHub (Dec 1, 2022): Oh huh, I wonder if the selection changes referenced here also caused #14464 and #14462 🤔
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#18313