Explicitly include '?' as a fallback key for keyboard shortcuts

(closes #5675) This allows for a localized version of the key, but iD will still respond to the '?'
openstreetmap · Jan 7, 2019 · 4ed2858 · 4ed2858
1 parent 826f66a
commit 4ed2858
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/data/shortcuts.json b/data/shortcuts.json
@@ -41,7 +41,7 @@
               "text": "shortcuts.browsing.help.help"
             },
             {
-              "shortcuts": ["shortcuts.toggle.key"],
+              "shortcuts": ["shortcuts.toggle.key", "?"],
               "text": "shortcuts.browsing.help.keyboard"
             },
             {

diff --git a/modules/ui/shortcuts.js b/modules/ui/shortcuts.js
@@ -21,7 +21,7 @@ export function uiShortcuts(context) {
 
 
     context.keybinding()
-        .on(t('shortcuts.toggle.key'), function () {
+        .on([t('shortcuts.toggle.key'), '?'], function () {
             if (d3_selectAll('.modal-shortcuts').size()) {  // already showing
                 if (_modalSelection) {
                     _modalSelection.close();