-
Notifications
You must be signed in to change notification settings - Fork 30.7k
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
Investigate and Improve Key bindings customisations experience #18095
Comments
@sandy081 Insider build has been my daily driver since its inception; as well as Windows Insiders. Anyways, I checked it out the keybinding editor view in today's build. It looks great! |
@sandy081 If I can make two suggestions...
edit: The terminal view covers the keybindings at the bottom of the list. I believe this may be a bug because its behavior is not consistent with the rest of the editor. |
@sgtoj Thanks for the feedback. The design of this editor is to be simple and show the available Keybindings in the system. Source column was added to help user to show where a particular keybinding is coming from. To separate User and Default keybindings was one of the initial designs. But as a first cut and with some initial feedback, above simple design was chosen. I will gather all the feedback and update/improve the design. Yes, terminal view covering the keybindings is a bug and I will fix it today and you can get it in tomorrows insiders. Thanks |
- Use List widget to render keybindings
- Open default keybindings while opening keybindings file like before
- Define keybinding action with keyboard shortcut - Search keybindings action with keyboard shortcut
- Register keybinding for remove action - Make other actions as just keybinding actions
- Down arrow to navigate from search to list
- Do not navigate by Enter from search box
-- Sorry for the English, I used Google Translator -- @sandy081 The new Keybindings screen would be possible or feasible, allow search not only for the column 'command', but also for the column 'Keybinding' and 'When'? Why this? VSCode is not yet in my language, and searching for the shortcut key may be faster in this case. I know we still have the old screen. |
@Tekbr Searching for keybindings is possible. Typing |
@sandy081 Oh, I was typing |
I will be improving Search to accept both full or short forms |
OK. Thanks for the quick response. |
The interface frozen here #23039 |
-- Sorry for the English, I used Google Translator -- I do not know if it's in a commit that was not made available in Insiders. But in 1.11.0 - Insiders d98a518 2017-03-29T06: 11: 14.271Z, I can not use Ctrl instead of using Control. Can you check it please? And not forgetting, thanks for allowing you to search the 'When' column. 😄 👍 |
- Opening keybindings - While performing actions
Investigate and improve key bindings customisations experience.
Following are the proposals, use cases, challenges in this area
Keybindings view
A table based view, each row for each key binding
Show the correct Keybindings based on the keyboard layout (both defaults and user)
Currently, keybindings show the keys those were shown in the US Keyboard. Since the keys location changes with Keyboard layout, some of the Keybindings shown as is in the file do not work. For eg:
ctrl+` does not work on German layout key board. Instead the associated Keybinding for this is ctrl+<
Show the invalid Keybindings
Some key bindings might not makes sense in some keyboard layouts since . Show such keybindings as unbound entries, probably with message?
Grouping
Group key bindings by VS code core and each extension key bindings
Command id vs title
Show just title or just id or both for the command column? Not all commands have labels and not all command ids are straight to interpret what action they perform. Not all command ids maps well with labels.
When context
when
is a combination of the location in VS Code where key binding is applied to and the pre-condition to be met to trigger the action. Not all commands have proper separation of location and precondition. Also some commands have multiple locations. How to render this when clause? How to let user edit this?Command args
args
How to display argsBound and unbound commands
Display bound and unbound commands as two different groups
As a user, I would like to associate a Keybinding to a command/action
User should be able to find an action say in
Keybindings editor
orCommand Pallette
and associate a key binding to it. Use the widgetDefine Keybinding
to record the Key bindings instead of writing it by hand.An action next to command pallette list entry to bind/unbind the key binding?
As a user, I would like to find the command/action for a Keybinding.
User should be able to find the Keybinding in
Keybindings editor
and see the associated commands/actions to it.As a sophisticated user, I would like to switch to native old keybindings json from the new editor
The text was updated successfully, but these errors were encountered: