Bug Report: Double Click Selection should span delimiter runs #3297

Closed
opened 2026-01-30 23:17:57 +00:00 by claunia · 2 comments
Owner

Originally created by @carlos-zamora on GitHub (Aug 8, 2019).

Environment

Caused by #1197.
More apparent in #2184.

Steps to reproduce

  1. ensure double click delimiters includes " "
  2. double click on empty line

Expected behavior

Selection should encompass entire line.

Actual behavior

Selection encompasses one cell.

Additional Info

This one's on me. Idk why I thought that double clicking a delimiter should only select the delimiter itself. That's not useful.

Instead, the selection seems like it should expand to the first non-delimiter in either direction (exclusive range). For example,
O is non-delimiter
X is delimiter
Double clicking on any of the X would result in the following selection (denoted by [ and ])
OOOXXXXXXOOO
OOO[XXXXXX]OOO

Originally created by @carlos-zamora on GitHub (Aug 8, 2019). # Environment Caused by #1197. More apparent in #2184. # Steps to reproduce 1. ensure double click delimiters includes " " 2. double click on empty line # Expected behavior Selection should encompass entire line. # Actual behavior Selection encompasses one cell. # Additional Info This one's on me. Idk why I thought that double clicking a delimiter should only select the delimiter itself. That's not useful. Instead, the selection seems like it should expand to the first non-delimiter in either direction (exclusive range). For example, `O` is non-delimiter `X` is delimiter Double clicking on any of the X would result in the following selection (denoted by [ and ]) `OOOXXXXXXOOO` `OOO[XXXXXX]OOO`
Author
Owner

@carlos-zamora commented on GitHub (Aug 8, 2019):

Not sure if I'll be able to get to this one on time. I'm hoping that the PRs referenced above provide enough guidance to get started. Feel free to tackle this on (just be sure to mention that here).

@carlos-zamora commented on GitHub (Aug 8, 2019): Not sure if I'll be able to get to this one on time. I'm hoping that the PRs referenced above provide enough guidance to get started. Feel free to tackle this on (just be sure to mention that here).
Author
Owner

@DHowett-MSFT commented on GitHub (Aug 8, 2019):

This looks like #2206 😉

@DHowett-MSFT commented on GitHub (Aug 8, 2019): This looks like #2206 :wink:
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#3297