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

Keybinding editor could be clearer #11341

Open
Tracked by #6900 ...
elsaco opened this issue Sep 27, 2021 · 3 comments
Open
Tracked by #6900 ...

Keybinding editor could be clearer #11341

elsaco opened this issue Sep 27, 2021 · 3 comments
Labels
Area-Settings UI Anything specific to the SUI Issue-Task It's a feature request, but it doesn't really need a major design. Product-Terminal The new Windows Terminal.
Milestone

Comments

@elsaco
Copy link

elsaco commented Sep 27, 2021

Windows Terminal version (or Windows build number)

Windows Terminal Preview Version: 1.11.2421.0

Other Software

No response

Steps to reproduce

  • open terminal settings
  • in "Actions" add a new command

Expected Behavior

  • new command edit working

Actual Behavior

new command edit not working

This is an example trying to add the first drop-down option "Break into the debugger". The intended key cord was "alt+shift+b". Notice how "a", "l", "t" are echoed in the entry box, then when trying to add "+" it just show "shift+plus":

command_action_edit.mp4
@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 Sep 27, 2021
@zadjii-msft
Copy link
Member

Ah okay I see what's going on here. That's not a text entry field, you're supposed to just press the key chord you want to bind. So as you're typing alt, it's showing those keys one at a time. I think we've got an open issue already for making that more clear - something @carlos-zamora can link if he likes.

@zadjii-msft zadjii-msft added Area-Settings UI Anything specific to the SUI Issue-Question For questions or discussion Resolution-Answered Related to questions that have been answered labels Sep 27, 2021
@zadjii-msft
Copy link
Member

I'm reopening this to track the discussion we had as a team:

  • this box should automatically stop being in "edit mode" as soon as a non-modifier vkey is pressed
  • When modifiers are pressed/released in edit mode, then the box should show ctrl+shift+, and update in realtime

that'll help make this clearer

@zadjii-msft zadjii-msft reopened this Sep 27, 2021
@zadjii-msft zadjii-msft changed the title Actions edit not working Keybinding editor could be clearer Sep 27, 2021
@zadjii-msft zadjii-msft added Issue-Task It's a feature request, but it doesn't really need a major design. Product-Terminal The new Windows Terminal. and removed Issue-Question For questions or discussion Resolution-Answered Related to questions that have been answered labels Sep 27, 2021
@ghost ghost removed the Needs-Tag-Fix Doesn't match tag requirements label Sep 27, 2021
@zadjii-msft zadjii-msft added this to the Terminal v2.0 milestone Sep 27, 2021
@DHowett
Copy link
Member

DHowett commented Oct 5, 2021

@lhecker and I had a discussion about this weeks ago, and I think we never filed an issue for it.

Thanks!

@zadjii-msft zadjii-msft removed the Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting label Oct 5, 2021
@zadjii-msft zadjii-msft modified the milestones: Terminal v2.0, 22H2 Jan 4, 2022
@zadjii-msft zadjii-msft modified the milestones: 22H2, Backlog Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Settings UI Anything specific to the SUI Issue-Task It's a feature request, but it doesn't really need a major design. Product-Terminal The new Windows Terminal.
Projects
None yet
Development

No branches or pull requests

3 participants