Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"shift extend selection" feature doesn't work in the click->shift-click case because single-cell selection is gone #5766

Closed
ymkun opened this issue May 6, 2020 · 11 comments · Fixed by #8611
Assignees
Labels
Area-TerminalControl Issues pertaining to the terminal control (input, selection, keybindings, mouse interaction, etc.) Help Wanted We encourage anyone to jump in on these. Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-3 A description (P3) Product-Terminal The new Windows Terminal. Resolution-Fix-Committed Fix is checked in, but it might be 3-4 weeks until a release.
Milestone

Comments

@ymkun
Copy link

ymkun commented May 6, 2020

Environment

Windows build number: [19041.208  wsl ]
Windows Terminal version (if applicable): 0.11.1251.0

Any other software?

Steps to reproduce

  1. Single click.
  2. Hold SHIFT and single click an unselected region.

Expected behavior

Region between two click should be selected.

Actual behavior

region between two click hasn't been selected.
I remeber this feature has been released in "Windows Terminal Preview v0.9.433.0", said: "You can now hold down Shift and click to extend your selection (#4404)". But in recent releases, it gone.

@ghost ghost added Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Tag-Fix Doesn't match tag requirements labels May 6, 2020
@DHowett-MSFT
Copy link
Contributor

So, fair enough. Shift only works to extend an actual extant selection Since single-click no longer selects a cell, it doesn't comprise an active selection.

We should probably expand the touchdown anchor when you click + shift-click. It's not high-priority, because there's a workaround (select 2 cells, then use shift-click.)

@DHowett-MSFT DHowett-MSFT added Area-TerminalControl Issues pertaining to the terminal control (input, selection, keybindings, mouse interaction, etc.) Help Wanted We encourage anyone to jump in on these. Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-3 A description (P3) Product-Terminal The new Windows Terminal. labels May 7, 2020
@ghost ghost removed the Needs-Tag-Fix Doesn't match tag requirements label May 7, 2020
@DHowett-MSFT DHowett-MSFT added Needs-Tag-Fix Doesn't match tag requirements and removed Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting labels May 7, 2020
@ghost ghost removed the Needs-Tag-Fix Doesn't match tag requirements label May 7, 2020
@DHowett-MSFT DHowett-MSFT added this to the Terminal Backlog milestone May 7, 2020
@DHowett-MSFT DHowett-MSFT changed the title "shift extend selection" feature gone "shift extend selection" feature doesn't work because single-cell selection is gone May 7, 2020
@DHowett-MSFT DHowett-MSFT changed the title "shift extend selection" feature doesn't work because single-cell selection is gone "shift extend selection" feature doesn't work in the click->shift-click case because single-cell selection is gone May 7, 2020
@ymkun
Copy link
Author

ymkun commented May 7, 2020

So, fair enough. Shift only works to extend an actual extant selection Since single-click no longer selects a cell, it doesn't comprise an active selection.

We should probably expand the touchdown anchor when you click + shift-click. It's not high-priority, because there's a workaround (select 2 cells, then use shift-click.)

Thanks. It's just a little inconvenient when select a long region now.

@Don-Vito
Copy link
Contributor

@DHowett, @carlos-zamora, @ymkun - ConEmu does something that is IMHO more intuitive:

  1. Shift+Click - selects single cell
  2. The next Shift+Click expands this selection.

ShiftSelect

If you like this approach, I can commit (it is less than one-liner 😊)

@DHowett
Copy link
Member

DHowett commented Dec 17, 2020

Sounds great, with one caveat.

We need to figure out how this works with Shift suppressing VT mouse mode. Will any single shift-click on a mouse mode application cause a single cell selection?

I might be okay with that.

@Don-Vito
Copy link
Contributor

@DHowett - I am not sure I understood the concern, can you please explain?

I mean I tried this in tmux with mouse mode on, and it works exactly like shift +drag one char (but with no need to drag).

Guess I am missing something.

@DHowett
Copy link
Member

DHowett commented Dec 18, 2020

Sure.

I was worried that introducing this behavior would make it difficult to single-click on the terminal itself when there was a mouse mode application running without selecting a single cell.

However, thinking about it more: There is no reason to click on the terminal itself during mouse mode unless you are making a selection, so this is okay. As long as it does not highlight a cell when you use Ctrl+Shift+Click on a URL!

@Don-Vito
Copy link
Contributor

Sure.

I was worried that introducing this behavior would make it difficult to single-click on the terminal itself when there was a mouse mode application running without selecting a single cell.

However, thinking about it more: There is no reason to click on the terminal itself during mouse mode unless you are making a selection, so this is okay. As long as it does not highlight a cell when you use Ctrl+Shift+Click on a URL!

I will try this. If works as expected - should I submit PR under this ticket?

@DHowett
Copy link
Member

DHowett commented Dec 18, 2020

Absolutely! 😄 We love your PRs.

@Don-Vito
Copy link
Contributor

Don-Vito commented Dec 18, 2020

@DHowett - lol. Actually I am asking if I should mark the PR as "Closing" this one.. or not. BTW - URL works like a charm

@DHowett
Copy link
Member

DHowett commented Dec 18, 2020

Yeah, I think that your proposal is sufficient to solve this issue!

@ghost ghost added the In-PR This issue has a related PR label Dec 18, 2020
@ghost ghost closed this as completed in #8611 Dec 18, 2020
@ghost ghost added Resolution-Fix-Committed Fix is checked in, but it might be 3-4 weeks until a release. and removed In-PR This issue has a related PR labels Dec 18, 2020
ghost pushed a commit that referenced this issue Dec 18, 2020
@ghost
Copy link

ghost commented Jan 28, 2021

🎉This issue was addressed in #8611, which has now been successfully released as Windows Terminal Preview v1.6.10272.0.:tada:

Handy links:

mpela81 pushed a commit to mpela81/terminal that referenced this issue Jan 28, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-TerminalControl Issues pertaining to the terminal control (input, selection, keybindings, mouse interaction, etc.) Help Wanted We encourage anyone to jump in on these. Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-3 A description (P3) Product-Terminal The new Windows Terminal. Resolution-Fix-Committed Fix is checked in, but it might be 3-4 weeks until a release.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants