Skip to content

Commit

Permalink
Fix default Mac shortcut
Browse files Browse the repository at this point in the history
From
https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/commands

On Macs, "Ctrl" is interpreted as "Command", so if you actually need
"Ctrl", specify "MacCtrl".
  • Loading branch information
danielhollas committed Dec 23, 2019
1 parent 0c8c7f8 commit 2e84c12
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion app.js
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@
$translateMathBtn = $('<button id="translate_math" tabindex="-1" class="btn btn-icon"><i class="static-icon-copy"></i></button>');
var shortcut;
if(isMac)
shortcut = " (Cmd+C)"
shortcut = " (Ctrl+C)"
else
shortcut = " (Alt+C)"
title = "Copy Source & Translate Math " + shortcut
Expand Down
4 changes: 2 additions & 2 deletions manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
"name": "Khan Academy Dots",
"description": "Translate math notation in Khan Academy Crowdin strings.",
"author": "Daniel Hollas & Szymon Bubak & Krzysztof Krystek & Robert Pala",
"version": "3.2.1",
"version": "3.2.2",
"permissions": [
"activeTab",
"storage"
Expand All @@ -23,7 +23,7 @@
"translate-math": {
"suggested_key": {
"default": "Alt+C",
"mac": "Ctrl+C"
"mac": "MacCtrl+C"
},
"description": "Click 'translate-math' button"
}
Expand Down

0 comments on commit 2e84c12

Please sign in to comment.