[1.16] Pressing enter with a selection no longer sends Enter #18405

Closed
opened 2026-01-31 06:12:58 +00:00 by claunia · 4 comments
Owner

Originally created by @DHowett on GitHub (Sep 9, 2022).

Possibly worth calling out in a relnote. I just stumbled over this because our default key binding for Enter is now Copy, and I didn't realize I had relied on Enter to send Enter even with a selection active.

Originally created by @DHowett on GitHub (Sep 9, 2022). Possibly worth calling out in a relnote. I just stumbled over this because our default key binding for Enter is now Copy, and I didn't realize I had relied on Enter to send Enter even with a selection active.
Author
Owner

@carlos-zamora commented on GitHub (Oct 4, 2022):

  • copyOnSelect should make Enter be ignored? Unbind it?

  • if copyOnSelect, copy keybindings should be passed through?

  • ctrl+c and Enter behave differently

    • see how we handle ctrl+c (used to be in TermControl, probably Interactivity now?), and make sure Enter (or really all kbds) are handled similarly (see _selectionNeedsToBeCopied)
  • compare Mark Mode to mouse selection with copyOnSelect

@carlos-zamora commented on GitHub (Oct 4, 2022): - `copyOnSelect` should make `Enter` be ignored? Unbind it? - if `copyOnSelect`, `copy` keybindings should be passed through? - ctrl+c and Enter behave differently - see how we handle ctrl+c (used to be in TermControl, probably Interactivity now?), and make sure Enter (or really all kbds) are handled similarly (see `_selectionNeedsToBeCopied`) - compare Mark Mode to mouse selection with `copyOnSelect`
Author
Owner

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

We cool just moving this one to a docs bug/? Haven't had complaints about this in the 1.16 preview timeframe yet.

See https://github.com/MicrosoftDocs/terminal/issues/618

@zadjii-msft commented on GitHub (Dec 2, 2022): We cool just moving this one to a docs bug/? Haven't had complaints about this in the 1.16 preview timeframe yet. See https://github.com/MicrosoftDocs/terminal/issues/618
Author
Owner

@ghost commented on GitHub (Dec 6, 2022):

This issue has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 4 days. It will be closed if no further activity occurs within 3 days of this comment.

@ghost commented on GitHub (Dec 6, 2022): This issue has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for **4 days**. It will be closed if no further activity occurs **within 3 days of this comment**.
Author
Owner

@DHowett commented on GitHub (Dec 6, 2022):

Preview is pretty small, so we should be careful about drawing non-crash conclusions. But, yes.

@DHowett commented on GitHub (Dec 6, 2022): Preview is pretty small, so we should be careful about drawing non-crash conclusions. But, yes.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#18405