[keybindings] align keybindings-widget with vscode

[vince-fugnitto]

Fixes #5544

The purpose of the PR is to align the behavior and look of the
keybindings-widget more closely with that of vscode.

Features present in the PR:

1. removed the command id column.
2. renamed the scope column to source.
3. display the command id for commands without a label.
4. re-order the columns to that of vscode (actions, command, keybinding, source).
5. prioritized the display of commands with keybindings before those without.
6. updated the documentation.

Signed-off-by: Vincent Fugnitto vincent.fugnitto@ericsson.com

[fangnx]

Works nicely. Like the jsdoc comments!

[vince-fugnitto]

@lmcbout

If I compare with old values, there are some some command missing: i.e. "Unstage all" + other commands.

I don't understand, unstage all is present.

Second, what about the commands which don't have a name, they used to show.

They are still displayed. If a command does not have a label it displays its id instead.

It looks like we only see the command having a keybinding defined for them.

That's untrue. If you look closely, commands with keybindings are displayed first, then those without are displayed.

[lmcbout]

looks Fine,
the commands without key bindings are there, I probably had a search criteria that was not clear while doing the testing
Thanks @vince-fugnitto

[vince-fugnitto]

Thanks @vince-fugnitto

No problem! :)

[vince-fugnitto]

I'll hold the PR for after the release to see if anyone else has comments.