After updating Monaco Editor some Keybindings has collision for Java LS

[musienko-maxim]

Che version

• latest
• nightly
• other: please specify

Steps to reproduce

• Create a workspace based on Java Maven stack. Open HelloWorld.java class
• Set cursor in 6:18 position (on .out) and invoke Ctrl+F12 shortcut
  Expected result: The Go to the definition command should be invoked. The System class should be opened.
  Actual result: Looks like instead of Go to definition command is invoked Go to implementation command

Runtime

• kubernetes (include output of kubectl version)
• Openshift (include output of oc version)
• minikube (include output of minikube version and kubectl version)
• minishift (include output of minishift version and oc version)
• docker-desktop + K8S (include output of docker version and kubectl version)
• other: (please specify)

Screenshots

Installation method

• chectl
• che-operator
• minishift-addon
• I don't know

Environment

• my computer
  • Windows
  • Linux
  • macOS
• Cloud
  • Amazon
  • Azure
  • GCE
  • other (please specify)
• other: please specify

Additional context

This problem can be depended on the Keybinding map, which has some duplicated Keybindings. See on screenshot. Also i supposed that this problem can be reproduced for other language servers because the KeyMap has other duplications.

[tsmaeder]

@musienko-maxim I think this issue should be opened against the theia repo.

[tsmaeder]

P2 because there is an easy workaround.

[tsmaeder]

This has nothing to do with Java: VS Code does not define any keybindings.

[tsmaeder]

I checked in theia master: go to def is f12, go to impl is <ctrl>f12

[tsmaeder]

FWIW: I tried reproducing this but was not able to get che to work on minishift

[svor]

Default keybinding was changed, so for now:

• ctrl+f12 - Go to Implementation
• ctrl+f11 - Go to Definition