Change default to Alt, do test both Alt and Accel

Author: krassowski

README.md

@@ -27,7 +27,7 @@ over the underlined code to see a more detailed message
 
 ### Jump to Definition
 
-Use the context menu entry, or <kbd>Ctrl</kbd> + :computer_mouse: to jump to definitions (<kbd>⌘</kbd> on Mac; customizable); use <kbd>Alt</kbd> + <kbd>o</kbd> to jump back
+Use the context menu entry, or <kbd>Alt</kbd> + :computer_mouse: to jump to definitions (you can change it to <kbd>Ctrl</kbd>/<kbd>⌘</kbd> in settings); use <kbd>Alt</kbd> + <kbd>o</kbd> to jump back
 
 ![jump](https://raw.githubusercontent.com/krassowski/jupyterlab-lsp/master/examples/screenshots/jump_to_definition.png)
 

atest/05_Features/Jump.robot

@@ -19,6 +19,7 @@ Python Jumps between Files
 
 Ctrl Click And Jumping Back Works
     [Setup]    Prepare File for Editing    Python    editor    jump.py
+    Configure JupyterLab Plugin    {"modifierKey": "Accel"}    plugin id=${JUMP PLUGIN ID}
     Wait Until Fully Initialized
     ${usage} =    Set Variable    a_variable
     ${sel} =    Set Variable    xpath:(//span[contains(@class, 'cm-variable')][contains(text(), '${usage}')])[last()]
@@ -34,6 +35,9 @@ Ctrl Click And Jumping Back Works
     Wait Until Keyword Succeeds    10 x    1 s    Cursor Should Jump    ${new}
     ${back} =    Measure Cursor Position
     Should Be Equal    ${original}    ${back}
+    Configure JupyterLab Plugin    {"modifierKey": "Alt"}    plugin id=${JUMP PLUGIN ID}
+    Click Element    ${sel}    modifier=ALT
+    Wait Until Keyword Succeeds    10 x    1 s    Cursor Should Jump    ${original}
     [Teardown]    Clean Up After Working With File    jump.py
 
 *** Keywords ***

atest/Variables.robot

@@ -45,6 +45,7 @@ ${CM CURSORS}     css:.jp-MainAreaWidget:not(.lm-mod-hidden) .CodeMirror-cursors
 # settings
 ${LSP PLUGIN ID}    @krassowski/jupyterlab-lsp:plugin
 ${COMPLETION PLUGIN ID}    @krassowski/jupyterlab-lsp:completion
+${JUMP PLUGIN ID}    @krassowski/jupyterlab-lsp:jump_to
 ${DIAGNOSTICS PLUGIN ID}    @krassowski/jupyterlab-lsp:diagnostics
 ${CSS USER SETTINGS}    .jp-SettingsRawEditor-user
 ${JLAB XP CLOSE SETTINGS}    ${JLAB XP DOCK TAB}\[contains(., 'Settings')]/*[contains(@class, 'm-TabBar-tabCloseIcon')]

packages/jupyterlab-lsp/schema/jump_to.json

@@ -9,7 +9,7 @@
       "title": "Modifier key",
       "type": "string",
       "enum": ["Alt", "Control", "Shift", "Meta", "AltGraph", "Accel"],
-      "default": "Accel",
+      "default": "Alt",
       "description": "Keyboard key which needs to be pressed with click to jump. The allowed keys are Alt, Control, Shift, Accel, and AltGraph. Accel corresponds to Control or Meta (⌘ Command on Mac). Linux user: Meta key is also supported. Safari users: Meta key is also supported, AltGraph is not supported. To see which physical keys are mapped, visit: https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/getModifierState"
     }
   },